[Freeipa-devel] [PATCH] admiyo-freeipa-0015 hashchange

[Rob Crittenden]

Adam Young wrote:
> On 08/25/2010 10:41 AM, Adam Young wrote:
>> On 08/25/2010 07:39 AM, Pavel Zůna wrote:
>>> On 2010-08-24 23:57, Adam Young wrote:
>>>> This is a fairly substantial patch that makes use of the hashchange
>>>> event to drive the site. It affcets just about all places where click
>>>> events are triggered.
>>>>
>>>
>>> Just started reviewing the patch a few moments ago. Just 2 notes in
>>> advance:
>>>
>>> 1) It doesn't apply after the <script> eng tag patch, that was ACKed,
>>> but not pushed yet.
>>>
>>> 2) I don't want to sound like a nitpicker, but you should do some
>>> tweaking to your text editor. It seems to leave trailing white spaces
>>> and more importantly, it converts 8 spaces to tabs!
>>>
>>> The only edits in details.js are the tab conversions, so there should
>>> be no conflict with my details patch when you remove them.
>>>
>>> Pavel
>>
>>
>> OK. I think I can deal with both of those issues. I'll repost the
>> patch after, to have the one for the record on the mailing list, but
>> also so you can confrium it works with your changes.
>>
>> _______________________________________________
>> Freeipa-devel mailing list
>> Freeipa-devel at redhat.com
>> https://www.redhat.com/mailman/listinfo/freeipa-devel
>
> removed whitespace
> updated the Makefile.am to include add.js
> applies on top of recent changes to other files in the directory.

This isn't working for me. I tried both with lite-server and a real 
deployment and all I see are the header and the Edit: line (users, 
groups, etc). Clicking on the different tabs doesn't seem to do anything.

rob