On Sat, May 17, 2008 at 06:09:19PM +0200, Justus Winter wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
The UI constructor creates a button for the fullscreen mode, but it is not added to anything:
// Make the full screen button (to be attached to the viewport later) // TODO: attach to the viewport FullScreen = new HighlightButton(0,0,15,15); [...]
Any idea why this was postponed? Should it be enabled now?
The idea is that there's a gadget in the viewport (i.e. a small button in the lower right corner that shows even when in fullwindow mode). This was to be used to enable/disable full window mode. Now, thinking twice, maybe is not a bad idea to hook a right-click fullscreen to this button. Leaving left-click for fullwindow mode. ...but for that, the gadget code needs to be enabled.
Also I noticed, that the way events are handled is somewhat lax with respect to modifier keys. The code just checks if ctrl is pressed, not if ctrl but none of the other modifiers are pressed. This makes ctrl-l equivalent with ctrl-shift-alt-l. I'd like to hear your opinions, is this a bug or a feature?
Bug I think. The UI was coded in a hurry to get it running. Lax parts need polish. -- Cheers Jorge.-