- Issue created by @Grevil
- ๐ฉ๐ชGermany Anybody Porta Westfalica
Yeah big +1 on this. Currently a wide table kills the form displays. Definitely the wrapper should be scrollable. We should ask @thomas.frobieter for a MR.
- @grevil opened merge request.
- ๐ฉ๐ชGermany Grevil
Took a first stab on this. @thomas.frobieter should finalize it!
- ๐ซ๐ทFrance lazzyvn paris
I think it can be report to admin theme, with bootstrap 5 admin theme it works fine
- ๐ฉ๐ชGermany Anybody Porta Westfalica
@lazzyvn no, this is not specific to admin theme, it's specific to paragraphs_table. We can't assume other containers should be horizontally scrollable.
I understand your special relationship with bootstrap 5 theme, but this in a very helpful module for anyone, also Gin users and other Drupal users. Please consisder that. And currently this is neither working for Gin nor for Claro, without implementing this. People should never be forced to use your bootstrap 5 theme to have an acceptable UX. In general modules should not rely on specific admin themes.
- ๐ฉ๐ชGermany Anybody Porta Westfalica
@grevil thanks, LGTM, let's wait for further feedback.
thomas.frobieter โ made their first commit to this issueโs fork.
OK, I've added some fixes, but it's a 'meh' solution. It is very likely that this will require many more fixes for other field types. Gin's help icons are also buggy inside the table. Anyway... an improvement, but far from perfect.
- ๐ซ๐ทFrance lazzyvn paris
Again you must fixed stylint css. If It doesnโt pass I canโt merge like last one. If you donโt know how to do just create patch I will do latter
-
lazzyvn โ
committed 4939a350 on 2.x
Issue #3521910: Make the table wrapper scrollable horizontally
-
lazzyvn โ
committed 4939a350 on 2.x