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

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.


<<inline: table-border.png>>

_______________________________________________
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/

Reply via email to