? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
14 Oct 2018

Pull Request for Issue #22529 .

Summary of Changes

Make sure there is a border bottom on all table header cells

It got removed somewhere????

avatar brianteeman brianteeman - open - 14 Oct 2018
avatar brianteeman brianteeman - change - 14 Oct 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Oct 2018
Category Administration Templates (admin)
avatar wilsonge wilsonge - close - 14 Oct 2018
avatar wilsonge wilsonge - merge - 14 Oct 2018
avatar wilsonge wilsonge - change - 14 Oct 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-10-14 21:29:20
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 14 Oct 2018

Apparently I removed it here. I'm sure there was a reason. But can't remember what now (and currently since my upgrade to Mojave this week localhost is broken). Will merge it for now and can figure out at a later point in time why.

avatar brianteeman
brianteeman - comment - 14 Oct 2018

Thanks

avatar wilsonge
wilsonge - comment - 14 Oct 2018

@brianteeman figured out why I removed it. Under the header there was inconsistent underlines. And whilst we can add the underscore under the checkbox - I didn't see the need for two single pixel lines so I decided to remove it.

image

avatar brianteeman
brianteeman - comment - 14 Oct 2018

and removing it produces the error you highlight

avatar wilsonge
wilsonge - comment - 14 Oct 2018

Other way around. Adding this line in produces it (because the checkbox headers consistently aren't a th element but a td and therefore aren't underlined.

avatar brianteeman
brianteeman - comment - 14 Oct 2018

this pr added the underline to the td and the th
without this line you get the error - see screenshots
image
image

with the line you dont get the error - see screenshots
image
image

Add a Comment

Login with GitHub to post a comment