Skip to content
This repository has been archived by the owner on Jan 1, 2021. It is now read-only.

Add fullscreen toggle button to solve some usability issues #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Mar 10, 2012

  1. dd fullscreen toggle button to solve some usability issues

    (eg http://trac.openstreetmap.org/ticket/4002)
    
    A few things to note:
    
    1. I haven't actually tested this, because my local environment doesn't have the
    real maximise/minimise frame. But I've stepped through the code, and it looks ok.
    2. The icon could be prettier.
    3. I'm making use of EditController.toggleSize() which was marked private. Someo
    ne wise should review this.
    4. There should probably be some vigorous debate any time a new toolbar icon is
    added.
    5. The button is always enabled, and doesn't change to match state. I tried to
    make it do that, but I don't know the Flex event stuff well enough. My attempts are
    included below. I'm committing the alternative state .svg in case it's useful,
    although it's currently unused.
    
    In potlatch2.mxml (button id="fullscreen")
    
                toggle="true"
    	    selected="{Application.application.theController.isMaximised()}"
    	    selectedUpIcon="@embed('embedded/fullscreen-toggle.svg')"
    	    selectedDownIcon="@embed('embedded/fullscreen-toggle.svg')"
                selectedOverIcon="@embed('embedded/fullscreen-toggle.svg')"
    	    EditController.isMaximised()
    
    [Bindable(event="is_maximised")]
    public function get isMaximised():Boolean {
      return maximised;
    }
    
    EditController.toggleSize() {
    ...
      dispatchEvent(new Event("is_maximised"));
    }
    stevage committed Mar 10, 2012
    Configuration menu
    Copy the full SHA
    265c90e View commit details
    Browse the repository at this point in the history