Skip to content
This repository has been archived by the owner on Aug 3, 2022. It is now read-only.

update small button size and deprecate the xs one #145

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

phndiaye
Copy link
Member

@phndiaye phndiaye commented Jun 23, 2022

What does this PR do?

Update the height of the button's size small and deprecate the xs one.

Related to : https://github.com/upfluence/backlog/issues/1705

What are the observable changes?

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated documentation
  • Properly labeled

Additional Notes

@phndiaye phndiaye requested review from JulienVannier66, Miexil and a team as code owners June 23, 2022 15:31
@phndiaye phndiaye requested review from sBenjamin270495 and removed request for a team June 23, 2022 15:31
@@ -353,15 +353,14 @@ as size modifiers.
// --------------------------------------------------

.upf-btn--small, .upf-btn--sm {
height: @input-size-sm;
height: 24px;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use spacing css-vars ?

less/base/_buttons.less Outdated Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants