[Freeipa-devel] [PATCH] admiyo-0221-action-panel-to-top-tabs

[Adam Young]

Again, have not yet run Selenium against this, so please do not push.  
There are conflicts between this version and some of edewata's patch.  
Additionally, there are some know issues with the rendering on the ACI 
pages which I'll iron out before this gets submitted for real.

This version solves Issues 1,2,4,5 (sort of) ,8,9,and 10 from below.


On 04/19/2011 08:59 PM, Adam Young wrote:
> Good points all.  Only one minoir quibble, in-line.
>
>
> On 04/19/2011 05:31 PM, Endi Sukma Dewata wrote:
>> On 4/19/2011 3:07 PM, Adam Young wrote:
>>> going to post this, but with a request to hold on pushing to the 
>>> repo. I
>>> have not yet tested against selenium, and suspect that it will break 
>>> all
>>> selenium test navigation.
>>
>> I've tested this patch with static data. There are some issues:
>>
>> 1. Incorrect data after switching tabs.
>>    Open Users tab, then click User Groups tab, all of the groups will
>>    be admin's. Refresh the page, it will show the correct data.
>>
>> 2. Missing third level tabs.
>>    Open HBAC tab, the Services & Service Groups are missing.
>>    Open SUDO tab, the Commands and Command Groups tabs are missing.
>>    Open Role Based Access Control, the Permissions and Privileges tabs
>>    are missing.
>>
>> 3. Default tab is not activated.
>>    Open Users->admin, the Settings tab is inactive. It should be bigger
>>    than the other tabs.
>>
>> 4. Inconsistent position of the action buttons.
>>    Open Users tab, observe the position of the Delete & Add buttons.
>>    Then click one of the users, the Reset & Update buttons move to
>>    the left.
>>
>> 5. Entity label (e.g. Users) should be used instead of entity name
>>    (e.g. user) as the page title (next to the buttons).
>>
>> 6. The page title (e.g. user) is too close to the tab groups (e.g.
>>    Settings). It needs some space between them.
>>
>> 7. The order of tab groups is not very intuitive.
>>    Open Groups tab, click one of the groups. The default tab group (i.e.
>>    Settings) is located between Member and Member Of. It would be
>>    better to put the default tab at the left most position.
>
> While in general your observations are spot on, I'd like to point out 
> that this is intentional, but that the goal will be to always open the 
> leftmost tab.  This will allow us to make entities that are primarily 
> containers, like netgroups, as well as dns zone, default to the page 
> that shows the most common use case: manage contained entities.
>
>>
>> 8. The if-then clause on entity.js:46 is unnecessary. There's no need
>>    to check spec.facet_group before assigning it to that.facet_group.
>>
>> 9. The assignment on entity.js:296 is unnecessary. Each facet has a
>>    reference to the entity, so the entity header can be accessed using
>>    that.entity.header.
>
>>
>> 10. IPA.entity_header() should take a spec object instead of attribute
>>     list for consistency.
>>
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-admiyo-0221-1-action-panel-to-top-tabs.patch
Type: text/x-patch
Size: 58358 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110421/d65e7f24/attachment.bin>