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.

Sign up for GitHub

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

Jump to bottom

fix(styles): message popover adjustments #4951

Merged
merged 2 commits into from
Nov 2, 2023
Merged

fix(styles): message popover adjustments #4951

merged 2 commits into from
Nov 2, 2023

Conversation

N1XUS
Copy link
Contributor

@N1XUS N1XUS commented Oct 31, 2023

relates #4912

  • Adjusted message popover parameters for arrow icon font size;
  • List item paddings;
  • List item title font metrics

@N1XUS N1XUS added the Bug Something isn't working label Oct 31, 2023
@N1XUS N1XUS added this to the Sprint 125 milestone Oct 31, 2023
@N1XUS N1XUS requested a review from a team October 31, 2023 07:03
@N1XUS N1XUS self-assigned this Oct 31, 2023
@netlify
Copy link

netlify bot commented Oct 31, 2023 •
edited
Loading

✅ Deploy Preview for fundamental-styles ready!

Name Link
🔨 Latest commit c60e464
🔍 Latest deploy log https://app.netlify.com/sites/fundamental-styles/deploys/6542107e786cac000869686f
😎 Deploy Preview https://deploy-preview-4951--fundamental-styles.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

mikerodonnell89
mikerodonnell89 previously requested changes Oct 31, 2023
View reviewed changes
Copy link
Member

@mikerodonnell89 mikerodonnell89 left a comment

Choose a reason for hiding this comment

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

see my comment above

@N1XUS N1XUS changed the title fix(styles): message popover adjustments [ci visual] fix(styles): message popover adjustments Nov 1, 2023
@N1XUS N1XUS merged commit f679653 into main Nov 2, 2023
8 checks passed
@N1XUS N1XUS deleted the fix/4912-message-popover branch November 2, 2023 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@InnaAtanasova InnaAtanasova InnaAtanasova approved these changes

@mikerodonnell89 mikerodonnell89 Awaiting requested review from mikerodonnell89

Assignees

@N1XUS N1XUS

Labels
Bug Something isn't working
Projects
None yet
Milestone
Sprint 125
Development

Successfully merging this pull request may close these issues.
3 participants
@N1XUS @mikerodonnell89 @InnaAtanasova