core icon indicating copy to clipboard operation
core copied to clipboard

UI: Update Base Font Size to 14px

Open fmontes opened this issue 1 year ago • 4 comments

Task description

As a dotCMS user, I want the base font size to be reduced to 14px while keeping the spacings based on 16px to improve readability and design consistency across the website.

### Acceptance Criteria
- [ ] The base font size in the UI should be 14px.
- [ ] Spacings between elements should remain consistent at 16px.
- [ ] All text should be legible and easily readable at the new font size.
- [ ] This change should NOT affect native a11y of the browser

External Links

N/A

Assumptions & Initiation Needs

  • Assumption: The current font size is set at 16px.
  • Initiation: Access to the dotCMS backend to make the necessary CSS changes.
  • We need to check ALL pages to make sure nothing breaks

Quality Assurance Notes & Workarounds

  • QA should check all pages to ensure the new font size is consistent and does not cause any readability issues.
  • In case of any spacing issues, the QA team should provide feedback for adjustments.

fmontes avatar May 31 '24 17:05 fmontes

IQA Failed

Tested in master

Most portlets and texts have the updated size, except these 3: Header: Keeps the old size Template & Containers: The text "Show archived" is too big, other portlets have the correct size Apps: The inside of Apps has the old size of 16px

The rest of the portlets have the correct size

https://github.com/user-attachments/assets/e7680aa3-d2fd-46d4-b23f-ffe4dcf178f3

KevinDavilaDotCMS avatar Aug 15 '24 18:08 KevinDavilaDotCMS

Internal QA: Failed

There are some fonts with the old size, specifically in the block editor. You could notice it in the following video:

https://github.com/user-attachments/assets/70eeb852-b0db-465b-ae6d-1305a0d28c1a

The test was run on: Browser: Chrome [Version 127.0.6533.120 (Official Build) (arm64)] Docker image: dotcms/dotcms:trunk_600b4f3

dcolina avatar Aug 20 '24 13:08 dcolina

Note to QA

The text inside the block editor (ProseMirror) remains at 16px base. If changes are needed, they will be handled in separate tickets. The same applies to text inside GraphQL query tool (CodeMirror) that remains at 13px.

valentinogiardino avatar Aug 26 '24 13:08 valentinogiardino

Internal QA: Needs work

There are a few sections that do not meet with the defined size. Specifically sections PERMISSIONS and RULES in general.

Image Image

Below we can find another sections where the font size must be fixed.

Image Image Image

dcolina avatar Aug 26 '24 17:08 dcolina

Internal QA: Passed.

It was tested on all the supported browsers (Chrome, Firefox, Edge) using the docker image dotcms/dotcms:trunk_dea1913.

dcolina avatar Sep 23 '24 14:09 dcolina

Fixed the font-size now is 14px Image

bryanboza avatar Sep 23 '24 20:09 bryanboza