View Issue Details

IDProjectCategoryView StatusLast Update
0022492mantisbtuipublic2017-10-08 23:52
ReporterKunda Assigned Tocommunity  
PriorityhighSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product Version2.2.0 
Target Version2.7.0Fixed in Version2.7.0 
Summary0022492: Regression: Resolved/Closed issues are not shown with a line-through (strike-through)
Description

0017920 is a closed issue and should be shown with a strike-through.

Steps To Reproduce

I discovered this through a downstream issue: https://freecadweb.org/tracker/view.php?id=2949
I verified this by opening 0022180 and looking at the summary

TagsNo tags attached.

Relationships

related to 0011721 new preferred setting at what moment the bug number becomes strikethrough 
has duplicate 0022834 closedvboctor Strikeout issue/ticket numbers in "Recently Visited" list at top of page if they are resolved 

Activities

dregad

dregad

2017-03-08 05:48

developer   ~0055984

Last edited: 2017-03-08 05:52

Resolved/Closed Issues are not striked-through in 2.x. This has been the case as far back as it goes (even in the early development versions of the modern-ui branch prior to 2.0.0-beta.1).

I didn't really pay attention to that until you mentioned it, but personally think it is a useful feature to visually indicate that an issue is closed, without having to hover with the mouse (the pop-up text does gives the status), so I would consider this a regression from 1.x that should be addressed.

I'm not really sure if this was by design, or just an overlooked side effect of migrating to the new UI; @syncguru or @vboctor, please comment.

Kunda

Kunda

2017-03-08 07:17

reporter   ~0055987

If it was by design, wouldn't have it been mentioned somewhere like in a changelog or ticket ?

dregad

dregad

2017-03-08 08:22

developer   ~0055990

If it was by design, wouldn't have it been mentioned somewhere like in a changelog or ticket ?

In an ideal world yes, but I'm afraid you're presuming too much of our development process ;)

Kunda

Kunda

2017-03-08 08:38

reporter   ~0055991

Well it's as easy as adding this to the CSS for it to work again:

.resolved {
text-decoration: line-through;
}

dregad

dregad

2017-03-08 09:00

developer   ~0055994

I know, but still I'd like to get the guys' input since I don't know if they removed it intentionally or not.

atrol

atrol

2017-03-08 09:33

developer   ~0055995

Not sure if you are aware that there is at least one place where we still strike through, it's the Roadmap

Striked through text is hard to read, maybe we should use it just for issue id links, but not for any description.

dregad

dregad

2017-03-08 10:10

developer   ~0055998

Last edited: 2017-03-08 10:11

there is at least one place where we still strike through, it's the Roadmap

This is a different case though - this is handled through custom functions, and not even with CSS [1].

Chris_Z

Chris_Z

2017-03-08 12:16

reporter   ~0056000

Another one and the same approach 2.

Kunda

Kunda

2017-03-09 08:27

reporter   ~0056014

FYI, we're going to implement the temp CSS fix (0022492:0055991) on the FreeCAD tracker until you guys decide how to tackle this issue. I'm stating that here so it causes no confusion when looking at the downstream bug and it looks 'Fixed' when it's not. Cheers and thanks for your efforts, MantisBT 2.x is a great achievement.

Kunda

Kunda

2017-03-20 07:44

reporter   ~0056125

Last edited: 2017-03-20 09:13

Not sure if you are aware that there is at least one place where we still strike through, it's the Roadmap
Striked through text is hard to read, maybe we should use it just for issue id links, but not for any description.

@atol personally the strikethrough doesn't make it difficult for me to read the roadmap. Maybe a way to deal with this is to add a mouse hover effect that would remove the strikethrough effect when hovering over the text ?

atrol

atrol

2017-03-24 04:53

developer   ~0056182

add a mouse hover effect that would remove the strikethrough effect when hovering over the text ?

Does not work that well if you print the roadmap ;-)

To repeat what @dregad wrote:
I'm not really sure if this was by design, or just an overlooked side effect of migrating to the new UI; @syncguru or @vboctor, please comment.

vboctor

vboctor

2017-04-01 17:09

manager   ~0056314

I'm personally not aware of this change being intentional. I agree with @atrol that we shouldn't strike through issue titles, but just issue ids. @syncguru did you drop this intentionally?

didaquis

didaquis

2017-04-01 21:00

reporter   ~0056320

It would be interesting to have a configuration parameter to decide if you want strike-through text for Closed tickets on the Roadmap page. Take in consideration the possibility of add a setting for that. It's not the first time users suggest a setting option. For example: 0011721

Kunda

Kunda

2017-05-17 07:36

reporter   ~0056878

I agree with @didaquis I don't mind the strike through on the roadmap page and you could make it disappear if you hover the mouse over it. As for this ticket even though this is low priority it looks like a regression so please consider fixing.
CC @syncguru

Padraig

Padraig

2017-07-14 06:14

reporter   ~0057224

Why does this not have a target version?

Kunda

Kunda

2017-07-27 06:12

reporter   ~0057338

Temp fix is:
<code>
.resolved {
/ Temp fix for MantisBT 0022492 /
text-decoration: line-through;
}
</code>
also https://github.com/FreeCAD/FreeCAD-Homepage/blob/454859fed8ad2aa7b606143b3e9bbc943a73f84a/tracker/plugins/FreecadTheme/freecad.css#L84-L87

Kunda

Kunda

2017-07-27 06:14

reporter   ~0057339

I meant that 0022492:0057338 is a temporary workaround fix till the mantisbt folks decide how they want to proceed 0011721

Kunda

Kunda

2017-09-22 15:40

reporter   ~0057797

Soft bump, wondering if the team has figured out this trivial issue yet?

syncguru

syncguru

2017-09-28 12:18

developer   ~0057836

I don't believe this was intentional. I don't have a strong opinion either way.

Related Changesets

MantisBT: master c368606e

2017-09-28 19:06

luzpaz

Committer: vboctor


Details Diff
Re-add strike-through on resolved issue id refs

Fixes 0022492
Affected Issues
0022492
mod - css/default.css Diff File