Edgewall Software
Modify

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#10100 closed defect (fixed)

Mixed case for ticket counts in roadmap

Reported by: Mitar Owned by: Christian Boos
Priority: normal Milestone: 1.0
Component: roadmap Version: 0.12.2
Severity: trivial Keywords: progress
Cc: mmitar@…
Release Notes:
API Changes:

Description

In roadmap, under the milestone, where ticket counts are, active and closed counts have lower-case (active, closed), while total count has first letter upper-case (Total). Probably this should be made to have matching case.

Attachments (1)

t10100-progress_bar-legend-r10661.diff (2.2 KB ) - added by Christian Boos 6 years ago.
simplify HTML for the progress bar legend.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 6 years ago by Remy Blank

Milestone: 0.13
Owner: set to Remy Blank

comment:2 Changed 6 years ago by Remy Blank

Resolution: fixed
Status: newclosed

Fixed in [10660] (one-character patch, yay!).

comment:3 Changed 6 years ago by Christian Boos

This got me thinking, what about an even more radical change?

Instead of a <dl> where we have to repeat the link for the <dt> and the <dd>:

Number of tickets:
closed:
54
active:
29
Total:
83

what about having something like:

Total of 83 tickets, among them: 54 closed, 29 active.

or simply:

83 tickets: 54 closed, 29 active.

comment:4 Changed 6 years ago by Mitar

This is then hard to process with Genshi to reformat/restructure. I would at least add some <span> elements around.

comment:5 Changed 6 years ago by Remy Blank

Resolution: fixed
Status: closedreopened

I don't really like the current state either. Your first suggestion (or something similar, like "83 tickets in total, split into: 54 closed, 29 active") is better, but may be difficult to translate (and requires at least a plural form). I find the second suggestion hard to read.

I can't find anything much better, though.

comment:6 Changed 6 years ago by Christian Boos

Indeed, one must be very careful with the i18n aspect here, as incidentally this part was one of the most difficult to get right during 0.12dev (#8588).

The second suggestion willl perhaps be more readable with <span>s and some extra whitespace, e.g.

83 tickets: 54 closed, 29 active.

But the above is certainly still prone to the i18n problems discussed in #8588.

So perhaps:

Total number of tickets: 83 closed: 54 active: 29

Not a big change from what we have now, but no <dl> and a single link, so I think it's better.

Also, I believe the construction “Total number of tickets: 1” is correct.

Changed 6 years ago by Christian Boos

simplify HTML for the progress bar legend.

comment:7 Changed 6 years ago by Remy Blank

Yes, that's quite good, actually. Is the hyphen intentional in the patch?

comment:8 in reply to:  7 ; Changed 6 years ago by Christian Boos

Replying to rblank:

Is the hyphen intentional in the patch?

Yes, that was a last minute addition. It suggests items in a list. Other symbols could be tried, for example:

Total number of tickets: 3 closed: 1 in progress: 1 new: 1

Total number of tickets: 3 - closed: 1 - in progress: 1 - new: 1

Total number of tickets: 3 · closed: 1 · in progress: 1 · new: 1

… and nothing is still an option, of course ;-)

Total number of tickets: 3 closed: 1 in progress: 1 new: 1

comment:9 in reply to:  8 Changed 6 years ago by Remy Blank

Replying to cboos:

Other symbols could be tried, for example:

No, the hyphen is fine, I was only wondering. So yes, please apply!

comment:10 Changed 6 years ago by Christian Boos

Keywords: progress added
Resolution: fixed
Status: reopenedclosed

Thanks! Committed in r10662.

comment:11 Changed 6 years ago by Remy Blank

Owner: changed from Remy Blank to Christian Boos

I don't stand a chance with my 1-character patch :)

comment:12 Changed 6 years ago by Christian Boos

Well, numerically you won, as I have a negative diffstat ;-)

(which is good, of course ;-) )

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted.
to The owner will be changed from Christian Boos 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.