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: win UI - feed claim modal ui & header readHistory ui #1019

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

Conversation

yusixian
Copy link
Contributor

@yusixian yusixian commented Oct 18, 2024

Description

  • Fix the bug that the close button of the authentication feed popup ui on authenticationwin is blocked by the top bar.

Before
image

After
image

  • Adjust the number of reading history avatars displayed on narrow screens
    Before
    image

After
image
image
image

I've adjusted a more comfortable spacing on win here, if it's not right on mac you can look at mac and then return the parameters based on the platform

Linked Issues

#958 (comment)

Additional context

Copy link

vercel bot commented Oct 18, 2024

@yusixian is attempting to deploy a commit to the RSS3 Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant