When there are no aliases the alias listing table (admin/config/search/path) shows one row with colspan 3 with "No URL aliases available. Add alias." while the header row has four columns - three columns and one col with colspan = 2.

CommentFileSizeAuthor
#4 728566-path-admin-columns-D7.patch3.53 KBDave Reid
#1 728566-path-admin-columns-D7.patch3.55 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
728566-path-admin-columns-D7.patch3.55 KB

Patch attached for review that makes this row generation much cleaner.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Version: 7.0-alpha2 » 7.x-dev
realityloop’s picture

realityloop
he/him
CreditAttribution: realityloop commented
Status: Needs review » Needs work

http://img.skitch.com/20100421-br5t3mnqq89yqwp8d239jc445x.png

needs to be colspan="4"
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs work » Needs review
FileSize
728566-path-admin-columns-D7.patch3.53 KB

Thanks for checking that! This one should address that. I forgot to remove the 'colspan => 2' on the operations column.

realityloop’s picture

realityloop
he/him
CreditAttribution: realityloop commented
Status: Needs review » Reviewed & tested by the community

Good to go!

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented

Can someone check if it fixes this issue: #746678: Markup issue with tables (admin/appearance/update)
I can test it later but it would be cool if someone can do it before me ;)

realityloop’s picture

realityloop
he/him
CreditAttribution: realityloop commented

Yes it fixes that

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Wow, that looks much nicer!

Committed to HEAD!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.