1014521 - Add edit mode behavior to gaia-header

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Comment 1

[Wilson Page [:wilsonpage]]

As discussed with Arnau I'd favour lower level configuration than edit-mode="true|false". It seems the functionality missing is the ability for the header to slide in/out when entering/leaving edit-mode.

In the Call Log the edit-mode header is a second header that is slid out of view by default. Upon entering edit-mode the header slides in to cover the primary header. The edit mode header looks something like:

<gaia-header action="cancel"></gaia-header>

With an added "show|hide" and "transition" configration, we give app developers everything they need to build their own 'edit-mode' header.

NEW ATTRIBUTES

<gaia-header action="cancel" transiton="slide" hide></gaia-header>

NEW PUBLIC API

header.show();
header.hide();
header.toggle();

I'm always in favour of lower-level more primitive configuration as it takes the burden off the component authors and increases flexibily/application, and therefore likelihood of being used.

From the app developer perspective I have seen developers not wanting to implement building blocks when/if they take away to much control. Let's make sure we don't do that :)

Comment 2

[Arnau March [:arnau] ( not working in Firefox OS anymore :( )]

Attachment: patch in github

Comment 3

[Wilson Page [:wilsonpage]]

Comment on attachment 8427068 [details] [review]
patch in github

Looks grand lad! Few nit-picks in Github, nothing major.

Rock on! :)

Comment 4

[Yan Or]

I'll defer to Wilson on the CSS.

Can you drop the "data-" prefix on the new show/hide and transition attributes?

Comment 5

[Arnau March [:arnau] ( not working in Firefox OS anymore :( )]

Merged: dadeed36538bf420612ece7619f8a423df1f7e02