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

fix: Billing Summary UI Regression from MUI Update #9622

Conversation

bnussman-akamai
Copy link
Member

Description 📝

Preview 📷

Before After
Screenshot 2023-09-01 at 10 24 48 AM Screenshot 2023-09-01 at 10 25 22 AM

How to test 🧪

  • Verify /account/billing looks good 🎨

@bnussman-akamai bnussman-akamai added Bug Fixes for regressions or bugs UX/UI Changes for UI/UX to review labels Sep 1, 2023
@bnussman-akamai bnussman-akamai self-assigned this Sep 1, 2023
Copy link
Contributor

@hana-akamai hana-akamai left a comment

Choose a reason for hiding this comment

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

Looks like on the mobile view there's some padding outside the boxes now vs before with no padding. I think it looks better with the padding, but also now there's some differences with Billing Contact, Payment Methods, and Billing & Payment History not having padding. Curious to hear the team's thoughts

Develop
before

This branch
after

@tyler-akamai
Copy link
Contributor

Other than the mobile view concern, looks great!

@bnussman-akamai bnussman-akamai changed the base branch from develop to release-v1.101.0 September 5, 2023 14:13
@bnussman-akamai bnussman-akamai merged commit 31498d0 into linode:release-v1.101.0 Sep 5, 2023
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fixes for regressions or bugs Ready for Review UX/UI Changes for UI/UX to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants