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)

Ticket522fix.patch (1.2 KB) - added by rmzelle 8 years ago.

Download all attachments as: .zip

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

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

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

(In [4880]) Closes #522, Arrow on full-screen button should point the other way when in full-screen mode

Note: See TracTickets for help on using tickets.