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

style(subscriptions-list):modify scrollbar style #1378

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

ni00
Copy link
Contributor

@ni00 ni00 commented Jul 28, 2023

PR Checklist

If you have any questions, you can refer to the Contributing Guide

What is the current behavior?

Different system scrollbar size is different, occupy different space, affect the display effect. (e.g. Win, Mac)

Issue Number

#1319

What is the new behavior?

  1. Set overflow to overlay and scrollbar to take up no space. overflow: overlay;
    refer link:MDN

  2. Uniform scrollbar styles.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

Are there any specific instructions or things that should be known prior to review?

Other information

image

@ni00 ni00 added fix Fix bug or issues desktop MQTTX Desktop labels Jul 28, 2023
@ni00 ni00 added this to the v1.9.5 milestone Jul 28, 2023
@ni00 ni00 requested a review from ysfscream July 28, 2023 05:17
@ni00 ni00 self-assigned this Jul 28, 2023
@ni00 ni00 linked an issue Jul 28, 2023 that may be closed by this pull request
@ni00 ni00 force-pushed the scroll-dev branch 2 times, most recently from 4fb2ae0 to fba8538 Compare July 28, 2023 05:42
@ysfscream ysfscream merged commit c847591 into emqx:main Jul 29, 2023
1 check passed
@ni00 ni00 deleted the scroll-dev branch July 31, 2023 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
desktop MQTTX Desktop fix Fix bug or issues
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[Bug] UI - Wrong format of sub topic column
2 participants