Edgewall Software
Modify

Opened 17 years ago

Last modified 9 years ago

#6783 new enhancement

Diff Tabular / Unified switcher highlights the wrong way around?

Reported by: osimons Owned by:
Priority: normal Milestone: next-major-releases
Component: version control/changeset view Version: devel
Severity: normal Keywords: css
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

When viewing a diff, the selected alternative is bolded (and not linked) and the alternative option that can be selected is greyed out (with link).

Shouldn't it be the other way around? It is very confusing, and contrary to common UI practice. Or could it be adjusted like the appearance of small mini tabs or something where both look like valid choices, but where current choice can still be clearly seen?

Attachments (3)

t6783-r6472-diff_switches.a.diff (1002 bytes ) - added by osimons 17 years ago.
Tabs in 5 minutes - still needs some adjustments.
tabs-01.jpg (7.2 KB ) - added by osimons 17 years ago.
Mini-tabs alternative.
t6783-r6472-diff_switches.b.diff (975 bytes ) - added by osimons 17 years ago.
Diff for mini-tab attempt.

Download all attachments as: .zip

Change History (9)

by osimons, 17 years ago

Tabs in 5 minutes - still needs some adjustments.

in reply to:  description comment:1 by Emmanuel Blot, 17 years ago

Replying to osimons:

Shouldn't it be the other way around? It is very confusing, and contrary to common UI practice. Or could it be adjusted like the appearance of small mini tabs or something where both look like valid choices, but where current choice can still be clearly seen?

Indeed: ticket:6709:2

by osimons, 17 years ago

Attachment: tabs-01.jpg added

Mini-tabs alternative.

by osimons, 17 years ago

Diff for mini-tab attempt.

comment:2 by osimons, 17 years ago

I am no designer, and come as far as I can with this. This is an illustration of current status, and diff is included for anyone feeling the urge to improve it:

Mini-tabs alternative.

comment:3 by osimons, 17 years ago

Milestone: 0.11.1
Owner: changed from Christian Boos to osimons

Just moving it onto the radar.

comment:4 by Christian Boos, 16 years ago

Keywords: css added
Type: taskenhancement

comment:5 by Christian Boos, 16 years ago

Milestone: 0.11-retriage0.13

comment:6 by Ryan J Ollos, 9 years ago

Owner: osimons removed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set. Next status will be 'closed'.
The owner will be changed from (none) to anonymous. Next status will be 'assigned'.

Add Comment


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