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

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.


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

removed whitespace
updated the 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.


Freeipa-devel mailing list

Reply via email to