Edgewall Software
Modify

Opened 21 years ago

Closed 20 years ago

Last modified 18 years ago

#267 closed enhancement (fixed)

Make external references visually distinct

Reported by: daniel Owned by: Jonas Borgström
Priority: low Milestone: 0.8
Component: wiki system Version: 0.6.1
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Christopher Lenz)

Links referencing outside trac should be visually distinct, like i other wikis.

This should be done in CSS only.

Attachments (0)

Change History (8)

comment:1 by daniel, 21 years ago

Summary: Make external references visuall distinctMake external references visually distinct

comment:2 by daniel, 21 years ago

Milestone: 0.8
Priority: normallowest
Severity: normalenhancement

comment:3 by Christopher Lenz, 20 years ago

Milestone: 0.80.9

Moving to 0.9.

comment:4 by cboos@…, 20 years ago

See the patch I attached to #234, which contains an 'external' class.

comment:5 by cboos@…, 20 years ago

Priority: lowestlow

… the relevant part being:

  • htdocs/css/trac.css

    === htdocs/css/trac.css
    ==================================================================
     
    240240a.missing:link,a.missing:visited { background: #fafaf0; color: #998 }
    241241a.missing:hover { color: #000; }
    242242
     243a.external:link,a.external:visited { color: #00b }
     244
    243245#content.wiki { line-height: 140% }
    244246.wikitoolbar {
    245247 border: solid #d7d7d7;
  • trac/WikiFormatter.py

    === trac/WikiFormatter.py
    ==================================================================
     
    304304            return '<a href="%s">%s</a>' % (self._href.wiki(match), match)
    305305
    306306    def _url_formatter(self, match, fullmatch):
    307         return '<a href="%s">%s</a>' % (match, match)
     307        return '<a class="external" href="%s">%s</a>' % (match, match)
    308308
    309309    def _fancylink_formatter(self, match, fullmatch):
    310310        link = fullmatch.group('fancyurl')
     
    319319        elif module_link:
    320320            return '<a href="%s">%s</a>' % (module_link, name)
    321321        else:
    322             return '<a href="%s">%s</a>' % (link, name)
     322            return '<a class="external" href="%s">%s</a>' % (link, name)
    323323
    324324    def _intertrac_link(self, intertrac):
    325325        href = self.env.get_config('intertrac', intertrac.upper() + '.href')

comment:6 by Christopher Lenz, 20 years ago

Description: modified (diff)
Milestone: 0.90.8
Resolution: fixed
Status: newclosed

Daniel has implemented this in [1068].

comment:7 by Juanma Barranquero, 20 years ago

Am I the only one who finds the new external link style a bit heavy-handed? Perhaps a simple color change would have sufficed (yes, I know I can customize to my heart's content, but still… :)

And, if definitely using an icon, wouldn't be better to have it *after* the link, à la MediaWiki?

comment:8 by cboos@…, 20 years ago

I like the new style, and didn't hesitate a split second

to drop my "color-only" change…

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.