Damian Krzeminski wrote: > Carolyn Beeton wrote: >> I've done a scrub of all screens to clean up tables, patch attached. It >> creates a new tableDiv for tables, with a CSS property that causes a >> horizontal scrollbar to be displayed if the columns get too wide to fit >> in the space. Thus tables never overwrite the QuickHelp (some settings >> screens with wide labels and values might still) >> > > I tried it and it looks really nice. Any reason why the tableDiv could not > be a class. It's defined as an ID now, and we can only have one element > with an ID on the page: that would limit us to one table per page. > > Also there is some small problem with left margin. Looks like the table > border is clipped. Do you see the same thing in your browser or is it just > my FF? (please see attached .png) > > >> I noticed these problems while checking each screen: >> - the Cert Authorities screen needs work, but is beyond my skills
http://code.sipfoundry.org/changelog/sipXecs/?cs=17498 I ended up not committing any changes to Cert Authorities screen: they were breaking the UI unit test. Still looking into tableDiv changes... > > Any takers on that? > >> - the Permissions screen seems to be missing a checkbox > > Are you referring to the fact that built-in permissions are not selectable > in Permission table? If so, this is by design. Admin cannot remove build in > permission. If you add a custom permission it'll get a checkbox for selection. > > > I'll commit the 'textarea' and 'fieldset' changes. And see if I can use the > class instead of ID. Help with the missing table border apreciated. > D. _______________________________________________ sipx-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipx-dev Unsubscribe: http://list.sipfoundry.org/mailman/listinfo/sipx-dev sipXecs IP PBX -- http://www.sipfoundry.org/
