Review: Needs Fixing code
The inline diff is really messed up so I am just looking at the paste.

in lib/lp/registry/templates/pillar-involvement-portlet.pt, I see invalid 
markup. A list must contain an li, and it can never contain a table. The ul is 
not needed since the table is a block element.

+    <ul tal:condition="view/configuration_links" style="padding-top: 1em">
+      <table style="width: 100%">

maybe this works
     <table style="padding-top: 1em; width: 100%"
       tal:condition="view/configuration_links">
-- 
https://code.launchpad.net/~bac/launchpad/lep-projconfig/+merge/30999
Your team Launchpad code reviewers is requested to review the proposed merge of 
lp:~bac/launchpad/lep-projconfig into lp:launchpad/devel.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : launchpad-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to