Comment on attachment 345282 New Card layout - Updated patch including the comments from Standard8
> <tabpanels id="abTabPanels" flex="1"> >+ <!-- ** Name Tab ** --> >+ <!-- The following vbox contains two hboxes top: Name/Email/Phonenumber bottom: Email prefs. --> >+ <vbox> >+ <!-- This hbox contains two vboxes left: Name/Email, right: Phonenumbers --> >+ <hbox index="name" flex="1">
I think the index/flex attributes should be on the <vbox> element above.
I think the business vbox also needs an index.
Otherwise r=me with those fixed. You'll need an sr for this patch as well.
Comment on attachment 345282
New Card layout - Updated patch including the comments from Standard8
> <tabpanels id="abTabPanels" flex="1"> Phonenumber bottom: Email prefs. -->
>+ <!-- ** Name Tab ** -->
>+ <!-- The following vbox contains two hboxes top: Name/Email/
>+ <vbox>
>+ <!-- This hbox contains two vboxes left: Name/Email, right: Phonenumbers -->
>+ <hbox index="name" flex="1">
I think the index/flex attributes should be on the <vbox> element above.
I think the business vbox also needs an index.
Otherwise r=me with those fixed. You'll need an sr for this patch as well.