Edgewall Software
Modify

Opened 12 years ago

Closed 12 years ago

#537 closed defect (fixed)

Redundant invisible "New Ticket" link in navigation bar

Reported by: Christopher Lenz Owned by: Christopher Lenz
Priority: low Milestone: 0.8
Component: general Version: 0.7.1
Severity: trivial Keywords: accessibility
Cc:
Release Notes:
API Changes:

Description

The navigation bar contains an invisible link to the "New Ticket" module, although a visible equivalent already exists:

<div id="navbar" class="nav">
 <ul>
  <li><a href="/mptrac/trac/wiki"  accesskey="1">Wiki</a></li>
  <li><a href="/mptrac/trac/timeline"  accesskey="2">Timeline</a></li>
  <li><a href="/mptrac/trac/roadmap" >Roadmap</a></li>
  <li><a href="/mptrac/trac/browser/" class="active">Browse Source</a></li>
  <li style="display: none"><a href="/mptrac/trac/newticket"
                    accesskey="7">New Ticket (Accessibility)</a></li>
  <li><a href="/mptrac/trac/report" >View Tickets</a></li>
  <li><a href="/mptrac/trac/newticket"  accesskey="9">New Ticket</a></li>
  <li><a href="/mptrac/trac/search"  accesskey="4">Search</a></li>
 </ul>
</div>

The invisible link is bound to accesskey 7, the visible link to accesskey 9. Because 9 is already bound to the "About Trac" link, only 7 works for accessing the "New Ticket" link.

Why not simply set the accesskey of the visible link to 7 and remove the invisible link?

Attachments (0)

Change History (2)

comment:1 Changed 12 years ago by Christopher Lenz

Milestone: 0.8
Owner: changed from Jonas Borgström to Christopher Lenz
Status: newassigned

comment:2 Changed 12 years ago by Christopher Lenz

Keywords: accessibility added
Resolution: fixed
Status: assignedclosed

Fixed in [939].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christopher Lenz.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christopher Lenz 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.