Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Label asterisk bug #10494

Closed
1 task done
siyanadicheva opened this issue Jan 8, 2025 · 2 comments · Fixed by #10533
Closed
1 task done

Label asterisk bug #10494

siyanadicheva opened this issue Jan 8, 2025 · 2 comments · Fixed by #10533
Assignees
Labels
bug This issue is a bug in the code Medium Prio released TOPIC RD

Comments

@siyanadicheva
Copy link

Bug Description

The asterisk of the label should be 1rem (16px). Right now it is 1.25rem (20px).

Image

Affected Component

Label

Expected Behaviour

The asterisk of the label should be 1rem (16px).

Image Image

Isolated Example

No response

Steps to Reproduce

...

Log Output, Stack Trace or Screenshots

No response

Priority

Medium

UI5 Web Components Version

https://sap.github.io/ui5-webcomponents/components/Label/

Browser

Chrome

Operating System

iOS

Additional Context

No response

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@siyanadicheva siyanadicheva added the bug This issue is a bug in the code label Jan 8, 2025
@unazko
Copy link
Contributor

unazko commented Jan 9, 2025

Hello @SAP/ui5-webcomponents-topic-rd,

As described by @siyanadicheva the font-size should result into 1rem for the asterisk when the required property is set on the ui5-label.
Instead of using the proper CSS parameter the font-size is set to 1.25rem

Best regards,
Boyan

@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This issue has been resolved in version v2.7.0-rc.0 🎉

The release is available on v2.7.0-rc.0

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug in the code Medium Prio released TOPIC RD
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

4 participants