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

Feat: Edition dropdown #546

Merged
merged 7 commits into from
Dec 1, 2024
Merged

Feat: Edition dropdown #546

merged 7 commits into from
Dec 1, 2024

Conversation

Readpato
Copy link
Member

@Readpato Readpato commented Dec 1, 2024

@Readpato Readpato self-assigned this Dec 1, 2024
@Readpato Readpato requested a review from TheJoin95 December 1, 2024 14:19
Copy link

netlify bot commented Dec 1, 2024

Deploy Preview for keen-banoffee-86ed50 ready!

Name Link
🔨 Latest commit 9e9a65c
🔍 Latest deploy log https://app.netlify.com/sites/keen-banoffee-86ed50/deploys/674c79a78218850009d5911f
😎 Deploy Preview https://deploy-preview-546--keen-banoffee-86ed50.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.

@Readpato Readpato added Site New feature or request for the site OSDay25 labels Dec 1, 2024
@TheJoin95
Copy link
Member

Did you check also the mobile version?

@Readpato
Copy link
Member Author

Readpato commented Dec 1, 2024

Yup, mobile version is also addressed in the PR

I decided against going for a dropdown in the menu, so I just added a sub category to the "Editions" title

EDIT: Updated the style, look at comment blow

image

@Readpato
Copy link
Member Author

Readpato commented Dec 1, 2024

Addressed again the mobile styles since they were bothering me

image

@TheJoin95
Copy link
Member

From desktop the link is working only if I click on the text and not if I'm clicking on the box

@TheJoin95
Copy link
Member

^^ Same in mobile

@Readpato
Copy link
Member Author

Readpato commented Dec 1, 2024

Good catch @TheJoin95, fixed it :)

@Readpato Readpato merged commit cb6d8da into 2025 Dec 1, 2024
4 checks passed
@Readpato Readpato deleted the feat/editions-dropdown branch December 1, 2024 15:46
@Readpato Readpato restored the feat/editions-dropdown branch December 1, 2024 15:47
@Readpato Readpato deleted the feat/editions-dropdown branch December 1, 2024 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OSDay25 Site New feature or request for the site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants