Opened 10 years ago
Closed 7 years ago
#522 closed defect (fixed)
Arrow on full-screen button should point the other way when in full-screen mode
| Reported by: | dstillman | Owned by: | dstillman |
|---|---|---|---|
| Priority: | trivial | Milestone: | 1.0.10 |
| Component: | interface | Version: | 1.0 |
| Keywords: | Cc: | rmzelle |
Attachments (1)
Change History (5)
comment:1 Changed 10 years ago by dstillman
- Milestone changed from 1.0 Beta 4 to 1.0 Final
Changed 8 years ago by rmzelle
comment:2 Changed 8 years ago by rmzelle
- Type changed from enhancement to defect
comment:3 Changed 8 years ago by rmzelle
- Cc rmzelle added
comment:4 Changed 7 years ago by dstillman
- Resolution set to fixed
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.
I think I got an easy fix for this ticket. I got it working in my local 1.5SP3.6 copy, but, as I'm rather a novice in this territory, it's probably a good idea to check the fix to see if it doesn't break other stuff.
In the current trunk, there seem to be two attributes that are used to indicate whether full-screen mode is toggled on or off: zotero-tb-fullscreen.zoterotop and zotero-tb-fullscreen.fullscreenmode*. The former attribute is specified only once in all the Zotero code in overlay.js, but that line doesn't result in the desired behavior (hence the ticket). The latter attribute is used to add a shadow to the full-screen button when in full-screen mode. As this shadowing does work, all I did was group the css-line that specifies the use of the down-arrow button with the lines specifying the shadow, making it dependent on the fullscreenmode-attribute. I also removed the line that set the value of zotero-tb-fullscreen.zoterotop, as after this change it isn't read anywhere anymore anyway.
https://www.zotero.org/trac/browser/extension/trunk/chrome/content/zotero/overlay.js?rev=4041#L132