Defect #23031
closed
CSS Wrong icon for details
Added by César DJ Caësar 9114 almost 8 years ago.
Updated over 7 years ago.
Description
Hello,
I'm currently developing a plugin for Redmine, and I saw that there may be a mistake regarding a class:
.icon-details { background-image: url(../images/zoom-in.png); }
I would have written:
.icon-details { background-image: url(../images/magnifier.png); }
Because "zoom-in" already exists here:
.icon-zoom-in { background-image: url(../images/zoom_in.png); }
And "magnifier" doesn't exist anywhere.
You should use .icon-magnifier which uses magnifier.png. .icon-details class is no longer used and should be removed.
Jean-Philippe Lang wrote:
You should use .icon-magnifier which uses magnifier.png. .icon-details class is no longer used and should be removed.
Merci Jean-Philippe, but I'm currently using version 3.2.1 (stable) of Redmine, and there is no .icon-magnifier. There is a -search-margnifier in responsive.css, though.
I may have missed something but I looked among all the files using:
grep -rnw '/usr/share/redmine/' -e "magnifier"
- Related to Patch #24578: Remove unused CSS class ".icon-details" added
- Category changed from Themes to UI
- Status changed from New to Closed
- Resolution set to Wont fix
Since I have created a issue #24578 to remove icon-details, I am closing this issue.
César Lizurey wrote:
Merci Jean-Philippe, but I'm currently using version 3.2.1 (stable) of Redmine, and there is no .icon-magnifier. There is a -search-margnifier in responsive.css, though.
.icon-magnifier was introduced in Redmine 3.3.0.
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by