- Issue created by @psebborn
- last update
almost 2 years ago 68 pass - @psebborn opened merge request.
- Status changed to Needs review
almost 2 years ago 2:18pm 11 May 2023 - π¬π§United Kingdom psebborn
Created a merge request to fix this issue: for all themes except Claro the styles will be applied but for Claro it's left with the theme defaults.
- Issue was unassigned.
- π¨π¦Canada nedjo
Thanks for noting the issue. Related: #3068461: Make Features UI theme-agnostic β . I wonder if the rules that are breaking in Claro are more leftovers from Seven that now can safely be removed instead of adding exclusions.
- π¬π§United Kingdom psebborn
I would probably agree with that; as Claro's now the default admin theme it displays fine if those styles are removed. I left them in so as to keep support for Seven but perhaps that's an issue for Seven rather than Features UI?
Happy to update the merge request and remove them if that is the most appropriate course? - last update
almost 2 years ago 68 pass - π¬π§United Kingdom psebborn
Updated the Merge Request now, removing those unnecessary styles as discussed
- Open on Drupal.org βCore: 9.5.x + Environment: PHP 7.4 & MySQL 5.7last update
almost 2 years ago Not currently mergeable. - @dave-reid opened merge request.
- last update
almost 2 years ago 68 pass - last update
almost 2 years ago 68 pass - Status changed to RTBC
almost 2 years ago 5:32pm 6 June 2023 - πΊπΈUnited States dave reid Nebraska USA
I tested this out locally and confirmed it resolved the issue for me.
- First commit to issue fork.
- last update
almost 2 years ago 68 pass -
matthand β
committed 41760662 on 5.0.x authored by
psebborn β
Issue #3359698 by psebborn: Fixed display of details element when using...
-
matthand β
committed 41760662 on 5.0.x authored by
psebborn β
- Status changed to Fixed
almost 2 years ago 5:39pm 6 June 2023 - πΊπΈUnited States matthand Riverdale Park, Maryland
Thanks for reviewing and testing community. This change looks ready to merge into 3.x-dev branch. Marking as fixed.
Automatically closed - issue fixed for 2 weeks with no activity.