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: dark mode tag selection and suggestion #3004

Merged

Conversation

xyl1t
Copy link
Contributor

@xyl1t xyl1t commented Feb 26, 2024

Hello, I am really enjoying this app so far! This is my first self hosted app and first PR :)
This PR fixes three issues:

1. Correct dark mode highlight color of tag in tag suggestion

Screenshot 2024-02-26 at 17 29 35

2. Fix incorrect position of tag suggestion when editing a memo and the memo is scrollable

Before After
Screenshot 2024-02-26 at 17 34 10 Screenshot 2024-02-26 at 17 35 30

3. Fix false trigger of tag suggestion

Fix scenario like the following:

1. Text #tag text
        ^ backspace here
2. Text#tag text
       ^ space
3. Text #tag text
        ^ tag suggestion opens

Issue: When editing a long memo and editing a tag somewhere where the user has
to scroll down to, the tag suggestions would be out of place (too far down)
because the scrolling wouldn't be taken into account.

Fix: Substract the suggestions div by the scroll amount.
Fixes this issue like this:
1. Text #tag text
        ^ backspace here
2. Text#tag text
       ^ space
3. Text #tag text
        ^ tag suggestion opens
@xyl1t xyl1t requested a review from boojack as a code owner February 26, 2024 16:41
Copy link
Collaborator

@boojack boojack left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for your contribution!

@boojack boojack changed the title Fix dark mode tag selection and suggestion fix: dark mode tag selection and suggestion Feb 26, 2024
@xyl1t xyl1t closed this Feb 26, 2024
@xyl1t xyl1t reopened this Feb 26, 2024
@boojack boojack merged commit 5526355 into usememos:main Feb 27, 2024
9 checks passed
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.

None yet

2 participants