Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#616 closed defect (fixed)

Modify ContentPane link processing to skip #anchor-style links

Reported by: anonymous Owned by: anonymous
Priority: high Milestone:
Component: Widgets Version: 0.3
Keywords: Cc: jeoffwilks@…
Blocked by: Blocking:

Description

Pages loaded in ContentPane have their anchor links modified so that they "break out" of the content pane and load a new page instead.

This proposed patch to source:/trunk/src/widget/html/ContentPane.js would prevent it from processing anchor tags:

case "href":
    if(attr[3].charAt(0)=='#'){   //skip anchor tags!
        fixedPath = attr[3];
    } else if(attr[3].search(/^(https?|ftps?|file|javascript|mailto|news):/)==-1){
        fixedPath = (new dojo.uri.Uri(url, attr[3]).toString());
    } else {
        fixedPath = attr[3];
    }
    break;

Change History (4)

comment:1 Changed 8 years ago by anonymous

  • Cc jeoffwilks@… added
  • Milestone set to 0.3release
  • Version set to 0.3

comment:2 Changed 8 years ago by anonymous

  • Owner changed from mumme to anonymous
  • Status changed from new to assigned

I'm happy to merge this. Have you submitted a CLA? http://dojotoolkit.org/icla.txt

comment:3 Changed 8 years ago by mumme@…

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed as of rev 3565.

Modified test_RemotePaths.html to include anchortags.

Thank you for finding this!

Regards
Fredrik Johansson

comment:4 Changed 7 years ago by anonymous

  • Milestone 0.3release deleted

Milestone 0.3release deleted

Note: See TracTickets for help on using tickets.