styleguide icon indicating copy to clipboard operation
styleguide copied to clipboard

Replace legacy bootstrap classes supported by FontAwesome

Open verdure-peuser opened this issue 3 years ago • 2 comments

Bug Report

Prerequisites

  • [X] Did you reproduce the problem on TYPO3 core master?
  • [X] Did you reproduce the problem on TYPO3 v11.5 LTS?

Description

FontAwesome provides CSS classes that align with the old Bootstrap v3 classes, which will cause troubles with the removal of FontAwesome.

If this patch https://review.typo3.org/c/Packages/TYPO3.CMS/+/75342 is applied, a few classes are not available anymore. For example pull-left

Steps to Reproduce

  1. Apply https://review.typo3.org/c/Packages/TYPO3.CMS/+/75342
  2. Open Styleguide > Typography

Expected behavior: Colors are aligned left to right

Actual behavior: [What actually happened] Each color is rendered in one line each.

Versions

dev-main 7b1ccd6

verdure-peuser avatar Aug 05 '22 20:08 verdure-peuser

Thanks for the bug report. If I'm not fully wrong, is this not fixed with the merged #342 ?

Just mentioning, FontAwesome is removed only in main, not v11. So I guess we do not need to change here anything for v11 style too.

sbuerk avatar Aug 07 '22 16:08 sbuerk

@sbuerk the MR doesn't fully adress it. Within /Resources/Private/Templates/Backend/Typography.html there are still some pull-left classes, leading to missaligend color "blocks".

ghost avatar Aug 12 '22 07:08 ghost

The "list-unstyled" styles are used in various places in the core and are provided by a bootstrap mixin.

Applicable locations: Build/Sources/Sass/backend.scss > Build/Sources/Sass/_minimal.scss > Build/node_modules/bootstrap/scss/mixins/_lists.scss

I couldn't find any other occurrences of Bootstrap classes in Typography.html. If applicable, these have now been removed.

minapok avatar Oct 30 '23 21:10 minapok

Thanks Jasmina. I hope it's ok to close here since we think this has been resolved meanwhile, at least in styleguide main/v13.

lolli42 avatar Oct 30 '23 21:10 lolli42