Skip to content

Fix for Issue #1#2

Open
mariel-delagarza wants to merge 2 commits intomainfrom
bug-fix/table-header
Open

Fix for Issue #1#2
mariel-delagarza wants to merge 2 commits intomainfrom
bug-fix/table-header

Conversation

@mariel-delagarza
Copy link
Copy Markdown
Member

Incrase value for "top" to make table header viewable on all screens when sticky.

@mariel-delagarza mariel-delagarza linked an issue Jan 24, 2025 that may be closed by this pull request
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 24, 2025

Deploy Preview for ackti-archive ready!

Name Link
🔨 Latest commit 6bc639c
🔍 Latest deploy log https://app.netlify.com/sites/ackti-archive/deploys/67991af927a69000089f75de
😎 Deploy Preview https://deploy-preview-2--ackti-archive.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Copy Markdown
Member

@urchykli urchykli left a comment

Choose a reason for hiding this comment

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

Thanks for looking into this Mariel! The table header is still under the options header a little bit on mobile and the options header is very squished on mobile as you can see in the screenshot. We should remove the arrows on mobile, as they aren't necessary to scroll horizontally and it will free up room for the rest of the items.
Screenshot 2025-01-24 at 4 42 30 PM

@mariel-delagarza
Copy link
Copy Markdown
Member Author

Refreshing this PR after we made the scroll button and sticky header updates

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.

Table header is covered by options header

2 participants