J4 Issue ?
avatar chmst
chmst
28 Apr 2019

Is your feature request related to a problem? Please describe.

The screen for Database fix is not accessible. It looks like this:

database-fix

Describe the solution you'd like

There is no need to show a warning or info alert when opening the screen.

There is plenty of place on this screen, even if there are three or four items in the table. The whole text which is now in the popover should be visible at once if the screen is opened.
The same for all other content which is now in popvers.

Additionally the language strings could be more positive. Instead of speaking about problems or "no problem", let's speak about "the state of your database" .. here native speakers are required.

avatar chmst chmst - open - 28 Apr 2019
avatar joomla-cms-bot joomla-cms-bot - change - 28 Apr 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 28 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Apr 2019
Labels Added: J4 Issue ?
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 28 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 28 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Apr 2019
Title
[4.0][a11y] Installer Database should not use popovers
[4.0] Installer Database should not use popovers
avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Apr 2019
Title
[4.0][a11y] Installer Database should not use popovers
[4.0] Installer Database should not use popovers
avatar franz-wohlkoenig franz-wohlkoenig - edited - 28 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Apr 2019
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Apr 2019
Labels Removed: ?
avatar franz-wohlkoenig franz-wohlkoenig - unlabeled - 28 Apr 2019
avatar brianteeman
brianteeman - comment - 28 Apr 2019

From the screenshot it looks like there are more than two problems.

For me I question what value there is in displaying the details of the problems.

avatar chmst
chmst - comment - 28 Apr 2019

Indeed there are several issues.
What I wanted to be resolved here are the popovers and the alert.

If the infomation provided in the popovers is correct is not in scope of tis issue.

avatar brianteeman
brianteeman - comment - 28 Apr 2019

If the information is of no value then it's an easy fix

avatar brianteeman
brianteeman - comment - 30 Apr 2019

If #24441 ever gets tested and merged then it will be easy to apply the same technique to solve the problem here

avatar richard67
richard67 - comment - 18 May 2019

@brianteeman As you closed PR #24441 in favour of PR #24892 which got merged, can this also be applied for the issue here? As far as I see it is something else. Anyway, the information in the tool tips of the installer database view are relevant in my opinion. So if there is a way to make them accessible or replace the popovers by some div like you did in other places, it should be done. If you have an idea on what to do but are too busy to make a PR, give me some input and I will try my best to do it.

avatar brianteeman
brianteeman - comment - 18 May 2019

@richard67 use the technique in #24899 as an example.

avatar richard67
richard67 - comment - 18 May 2019

@brianteeman Not sure if I understand #24899 . How can I test that?

avatar brianteeman
brianteeman - comment - 18 May 2019

apply the pr. open an article for editing. use the browser back button to go back to the list view and checkout the change on the padlock icon

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Jun 2019

Closed as having Pull Request #25207

avatar franz-wohlkoenig franz-wohlkoenig - change - 15 Jun 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-06-15 12:21:18
Closed_By franz-wohlkoenig
avatar franz-wohlkoenig franz-wohlkoenig - close - 15 Jun 2019

Add a Comment

Login with GitHub to post a comment