- Issue created by @peterbihari
- ππΊHungary peterbihari
The CSS change in this patch solves the above mentioned layout issue in Claro.
- Status changed to Needs review
over 1 year ago 12:59pm 9 November 2023 - ππΊHungary peterbihari
The previous patch does not work. I upload a new patch complete with the missing .css file.
- π¨π¦Canada francismak
For our case, we were having a very long URL in the tag. When clicking the 'source' button, the layout break. It only happens to two columns layout in claro.
Unfortunately, the patch above not working in our particular case. Attached our patch for reference.
Tested in Drupal 10.1.4. - π¬π§United Kingdom Baysaa
Reroll patch for Drupal 10.3 for composer patches. `node-add.css` is no longer there in 10.3 so moved the css to layout.css
- π·πΈSerbia miksha
Last patch in https://www.drupal.org/project/claro/issues/3400204#comment-15655237 π Long string breaks the layout Needs review covers only SOURCE editor. It can be extended to cover default editor too.
- π·πΈSerbia miksha
Extended patch to cover both default editor and source editor against 10.x where node-add exists is in this comment.
- π·πΈSerbia miksha
Patch against 11.x that is targeting class
layout-region--main
.
- π·πΈSerbia miksha
Patch against 11.x that is targeting class "layout-region--main". Previous one was bad.
- π©π°Denmark ressa Copenhagen
Moving issue to Drupal core, per the project page:
Claro is now part of Drupal core
All new issues should be filed in the Drupal Core issue queue β .
- πΊπΈUnited States smustgrave
Issues should be against 11.x
Will need an MR and an issue summary update
unable to reproduce can u add a screenshot so it will be helpful