The screen for Database fix is not accessible. It looks like this:
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.
Labels |
Added:
?
|
Labels |
Added:
J4 Issue
?
|
Title |
|
Title |
|
Status | New | ⇒ | Discussion |
Labels |
Removed:
?
|
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.
If the information is of no value then it's an easy fix
@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.
@richard67 use the technique in #24899 as an example.
@brianteeman Not sure if I understand #24899 . How can I test that?
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
Status | Discussion | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-06-15 12:21:18 |
Closed_By | ⇒ | franz-wohlkoenig |
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.