?
avatar coolcat-creations
coolcat-creations
4 Sep 2018

Migrated issue from: joomla/40-backend-template#432

system settings - update

avatar coolcat-creations coolcat-creations - open - 4 Sep 2018
avatar brianteeman
brianteeman - comment - 4 Sep 2018

This is very hard to make accessible as you have a second row with content spanning multiple cells

avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Sep 2018
Title
[Joomla 4.0 backend template] System Settings - Update Extensions #432
[4.0] [Backend Template] System Settings - Update Extensions #432
avatar joomla-cms-bot joomla-cms-bot - unlabeled - 4 Sep 2018
avatar joomla-cms-bot joomla-cms-bot - unlabeled - 4 Sep 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Sep 2018
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Sep 2018
Category Templates (admin)
avatar coolcat-creations
coolcat-creations - comment - 4 Sep 2018

You can span the row with css grid and/or css does not have to span multiple cells codewise

avatar brianteeman
brianteeman - comment - 4 Sep 2018

Wrote a long answer but I see @mbabker has written it so much better and shorter

avatar coolcat-creations
coolcat-creations - comment - 4 Sep 2018

No, I meant there are solutions to make it semantically correct AND have a correct view. I doubt that it’s not possible at all.

avatar coolcat-creations
coolcat-creations - comment - 4 Sep 2018

Again I wonder why this comment is made like 11 months later...

avatar brianteeman
brianteeman - comment - 4 Sep 2018

maybe we thought it would be a lot less than 11 months before it would reach this repo

avatar brianteeman brianteeman - change - 4 Sep 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 4 Sep 2018
avatar brianteeman
brianteeman - comment - 4 Sep 2018

Pretty certain that the html specification is that the data in a table cannot span two table rows

avatar brianteeman
brianteeman - comment - 4 Sep 2018

Actually I am saying it is impossible

avatar zero-24 zero-24 - change - 4 Sep 2018
Labels Added: ?
avatar zero-24 zero-24 - labeled - 4 Sep 2018
avatar brianteeman
brianteeman - comment - 4 Sep 2018

It is a table of tabular data so it must be a table. And if we really are serious about accessibility and not just listening when it is convenient you cannot create a table of data in any other way.

avatar coolcat-creations
coolcat-creations - comment - 4 Sep 2018

Could be solved for example with a nested table where the URL is the tableheader. Screenreaders can read nested tables.

avatar coolcat-creations
coolcat-creations - comment - 4 Sep 2018
avatar brianteeman
brianteeman - comment - 4 Sep 2018

Nested tables are horrendous for accessibility

We already have row headers (maybe you should read the code) but they don't resolve the issue you are presenting

avatar C-Lodder
C-Lodder - comment - 5 Sep 2018
  • Pushing last cell to new line: You'd have to set each cell to inline-block and the last cell to block, then fix all the alignments and borders. Way too dirty, so don't even try it.
  • CSS Grid: Not even semantically correct for a data set
  • Nested tables: Please don't even go there

I'd probably go as far as saying the URL's aren't even needed, else simply have a button in the last cell that directs to the authors' site or something.

avatar brianteeman
brianteeman - comment - 5 Sep 2018

Because it is possible for any extension to change the value of the update server of a different extension it is important that the url is displayed and not hidden behind a button for security purposes. At least if it is visible you might notice that you are being directed to the wrong site to grab a malicious update

avatar C-Lodder
C-Lodder - comment - 5 Sep 2018

@brianteeman fair point. Just bung it below the extension name then:

screeny

avatar brianteeman
brianteeman - comment - 5 Sep 2018

Just like it is right now ;)

avatar alikon
alikon - comment - 5 Sep 2018

we can change the update site, if we want, from gsoc17 #17553

avatar C-Lodder
C-Lodder - comment - 5 Sep 2018

@brianteeman Shows the last time I did a git pull then

avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Apr 2019
Labels Removed: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - unlabeled - 4 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Apr 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 5 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Apr 2019
Title
[4.0] [Backend Template] System Settings - Update Extensions #432
[4.0] System Settings - Update Extensions #432
avatar franz-wohlkoenig franz-wohlkoenig - edited - 19 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Apr 2019
Labels Removed: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - unlabeled - 20 Apr 2019
avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-08-29 15:03:39
Closed_By joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 29 Aug 2019
avatar alikon alikon - change - 29 Aug 2019
Closed_Date 2019-08-29 15:03:39 2019-08-29 15:03:40
Closed_By joomla-cms-bot alikon
avatar joomla-cms-bot
joomla-cms-bot - comment - 29 Aug 2019

Set to "closed" on behalf of @alikon by The JTracker Application at issues.joomla.org/joomla-cms/22003

avatar alikon
alikon - comment - 29 Aug 2019

related to the old template, open a new issue if needed


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22003.

Add a Comment

Login with GitHub to post a comment