Closed Bug 756438 Opened 13 years ago Closed 13 years ago

Implement AppBar scripting interface and build AppBar content dynamcally

Categories

(Core :: Widget: Win32, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jimm, Assigned: bbondy)

References

Details

(Keywords: feature, Whiteboard: completed-elm [metro-mvp])

Currently we have static AppBar markup embedded in platform code. This should probably be replaced with dynamic generated content controlled through content via a scriptable interface. I think having default buttons would be ok, but resources like button labels and layout need to be controlled by front end code. http://mxr.mozilla.org/projects-central/source/elm/widget/windows/winrt/MainPage.cpp
Initial implementation: http://hg.mozilla.org/projects/elm/rev/35da5ac3cdbb Details: - I created a new XBL control called appbar, you can manually add items to it, typically toolbarbuttons. - I used that appbar control inside browser.xul. - I used the icons provided by shorlander for the appbar - It exposes a method for show() and dismiss(), and later we'll add more. - It automatically takes care of auto-dismissing when you click outside of the appbar. - The menubutton isn't working yet, because xul menus aren't working correctly in Metro - I just added all buttons for now in an arbitrary order, I'll need to fix that later to be to spec.
windows button + Z will show the appbar at any time, I landed with it showing originally but I'll remove that soon with some other small changes.
Added events for showing/hiding and registered for it. Inside the handler I use the SecondaryTile::Exists API but it has a bug in it currently. So as of right now detection doesn't work for if a tile is pinned. IE has the same problem with detection right now.
Whiteboard: completed-elm
Assignee: nobody → netzen
Keywords: feature
Whiteboard: completed-elm → completed-elm [metro-mvp]
elm -> m-c
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Depends on: 854269
Depends on: 841228
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.