Bug #55773
closedConsistent look for the element browser in RTE and link input fields
0%
Description
Link browser called from the RTE (BrowserLinks.php) looks differently than link browser triggered from any link input field (implemented in the ElementBrowser.php).
See attached screenshot - left side is RTE, right, link input field.
This situation is caused by huge amount of duplicated code between these 2 classes.
Also attached screenshot with the class diagram.
We should eliminate duplicated code and make both element browser look the same.
Files
Updated by Stanislas Rolland over 10 years ago
- Category set to RTE (rtehtmlarea + ckeditor)
Updated by Stanislas Rolland over 10 years ago
Tymoteusz Motylewski wrote:
Link browser called from the RTE (BrowserLinks.php) looks differently than link browser triggered from any link input field (implemented in the ElementBrowser.php).
See attached screenshot - left side is RTE, right, link input field.This situation is caused by huge amount of duplicated code between these 2 classes.
Also attached screenshot with the class diagram.We should eliminate duplicated code and make both element browser look the same.
Not so simple. The RTE has many additional configuration options and fields for this form. But if you manage to remove some redundant code, that would be nice.
I agree that the look of the forms in the RTE is bad. This was fixed in related issue.
Whether the form should be presented above or below the trees has been discussed recurrently in the past. I would not change this without some consensus.
Updated by Tymoteusz Motylewski over 10 years ago
Thanks Rolland for taking care! Can you point me to any discussion about it.
I think backend should look consistent, so users can learn it quicker, so we should decide on one position of the form.
What I've experienced is that having form above tree is better, as most beginners don't know that they need to scroll down to set link title or sth else.
I'm working on cleaning up duplicated code in these classes (starting from init()) I'll push sth soon.
Updated by Mathias Schreiber over 9 years ago
- Status changed from New to Needs Feedback
- Assignee set to Mathias Schreiber
Hey Tymoteusz,
any progress here?
Updated by Tymoteusz Motylewski over 9 years ago
the current status is that big parts of the refactoring already made it's way into 6.2 (e.g. streamlining the init()) part.
We still need to streamline the views. e.g. Fields should go above the tree, to make them easier to find.
Also there is still a lot of copy paste code, and html and js code inside php.
The current blocker is the lack of concept, how the link/image/element browser should look and behave like.
In RTE the wizard opends in the modal (ext-js based), but for link fields it opens a new window.
some quotes from core-dev slack:
Benni Mack:
----------
no need to take on these things now. I think we need a good UI concept to get rid of them. I think the UX guys should talk about that at the UX week.
I had an idea for the element browser things... have a suggest field, which searches the different things all for one, and then a layer next to the field.
Benjamin Kott:
-------------
we had agreed that modals should only be used for small information and confirmations
the master plan for those popups is still missing - but would be nice if they would leave
they will be problematic as soon as we reach a state where we can truly use the backend on a smartphone/tablet
but i think we need to develop some kind of "magic" simplified wizards that can append to the fields and do the job without the need of big communication in modals or overlays
So what we can do for now, is to e.g. try to make the wizards look more alike (like moving fields to top). But the big changes are blocked by lack of final solution for modals.
Updated by Alexander Opitz about 9 years ago
5 Month later, I'd like to know the state of this issue, is there help needed?
Updated by Tymoteusz Motylewski about 9 years ago
Definitely this is still an issue :)
Updated by Alexander Opitz about 9 years ago
- Status changed from Needs Feedback to New
- Assignee changed from Mathias Schreiber to Stanislas Rolland
I'm assigning it to Stanislas.
Updated by Riccardo De Contardi over 8 years ago
- Target version set to Candidate for Major Version
- TYPO3 Version changed from 6.2 to 8
No more for 6.2, I guess. That's why I switch version to 8.
Updated by Riccardo De Contardi over 7 years ago
- File typo3-linkmodals.jpg typo3-linkmodals.jpg added
I attach a screenshot with the situation on v. 8.7-dev (latest master)
On the left: all the link types (I used header > link); on the right: the modal for CKEditor
The last row is a linkhandler example.
Updated by Tymoteusz Motylewski over 7 years ago
- Status changed from New to Resolved
Updated by Benni Mack almost 6 years ago
- Status changed from Resolved to Closed