[Freeipa-devel] [PATCH] shorten-url
Adam Young
ayoung at redhat.com
Tue Jun 28 21:57:04 UTC 2011
On 06/28/2011 05:45 PM, Adam Young wrote:
> On 06/28/2011 04:51 PM, Adam Young wrote:
>> On 06/28/2011 04:18 PM, Endi Sukma Dewata wrote:
>>> On 6/28/2011 2:33 PM, Adam Young wrote:
>>>> On 06/28/2011 03:13 PM, Adam Young wrote:
>>>>> One known issue: uses the wrong style for automount tabs, leaving
>>>>> excess white space
>>>
>>> Some issues:
>>>
>>> 1. Reloading the some pages will bring you to a different page (i.e.
>>> bookmarking wouldn't work). Try reloading these pages:
>>> - Groups details/association facet
>>> - HBAC Rule search facet
>>> It looks like the state JS variable should be initialized with
>>> values from the URL.
>>>
>>> 2. The following statement in navigation.js:123:
>>>
>>> key2.search('^'+entity)
>>>
>>> might match more keys than we want (e.g sudo will match sudorule,
>>> sudocmd, sudocmdgroup). It might be better to do the matching twice,
>>> one for exact <value> matching and the other for <value>-* prefix.
>>>
>>> 3. I haven't verified this, but the removeAttribute() is supposed to be
>>> used with DOM elements. To remove a JS object property we should
>>> use:
>>>
>>> delete state[key];
>>>
>>> 4. As discussed over IRC, the logic for calculating tab depth assumes
>>> that only the leaf nodes can be hidden. This can be slightly
>>> improved
>>> by moving this code
>>>
>>> if (tab.hidden) {
>>> depth = depth -1;
>>> }
>>>
>>> from navigation.js line 258 into 253. This way any hidden tabs along
>>> the tab hierarchy will not be counted toward depth.
>>>
>>> 5. There's a whitespace warning.
>>>
>>
>>
>> _______________________________________________
>> Freeipa-devel mailing list
>> Freeipa-devel at redhat.com
>> https://www.redhat.com/mailman/listinfo/freeipa-devel
>
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110628/1981a0b7/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-admiyo-0255-6-shorten-url.patch
Type: text/x-patch
Size: 22452 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110628/1981a0b7/attachment.bin>
More information about the Freeipa-devel
mailing list