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

Issue: 48 when you scroll down or explore through the entries, the in… #69

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shubhammantri1
Copy link

…put clears

Issue: #48

Modifications: Implemented the solution from https://virtuoso.dev/top-items to address the problem. This modification ensures that the 'edit post box' div remains fixed in place, maintaining its visibility even when scrolling. Furthermore, it was necessary to eliminate the z-index from both the 'ball' and 'time' elements within a post, as they were previously overlaying the pinned post box.

image

Also, added 2px margin-top to fix the layout issue.

image

@shubhammantri1
Copy link
Author

Hi @UdaraJay ,
Please review and merge this PR. I have raised this PR to fix this issue #48.

@shubhammantri1
Copy link
Author

Hi @UdaraJay
Did you get a chance to review the PR?

@UdaraJay
Copy link
Owner

hey! thanks for contributing!

the editor should only be sticking to the top if it has content- if you can make it work that way I'll merge, otherwise I'll aim to add and merge down the line.

@shubhammantri1
Copy link
Author

hey! thanks for contributing!

the editor should only be sticking to the top if it has content- if you can make it work that way I'll merge, otherwise I'll aim to add and merge down the line.

Sure, I'll make the changes.

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.

2 participants