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

New Script: Header Blogs #844

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

marcustyphoon
Copy link
Collaborator

@marcustyphoon marcustyphoon commented Nov 20, 2022

Feel free to close. But hey, just in case.

Description

Implements the blog-icons-in-the-header functionality from legacy XKit's Header Options.

  • Highlight currently selected blog
  • Maybe add a hover text thingy, but also maybe not, who cares
  • Add count preference probably
  • soft navigation? :3

Testing steps

Comment on lines 11 to 15
"maxBlogs": {
"type": "text",
"label": "Maximum blogs",
"default": "100"
}
Copy link
Owner

Choose a reason for hiding this comment

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

pretty sure you can use flex-wrap: wrap; overflow: hidden; to simply fill the available space cleanly

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I am! But I think many people like putting like 3-5 blogs up there even if they have many more. (Legacy XKit defaulted to 3.) I personally don't want all 20+ blogs my main has access to up there if I fullscreen on a big monitor.

@marcustyphoon marcustyphoon marked this pull request as ready for review February 4, 2023 13:24
@AprilSylph AprilSylph closed this Feb 4, 2023
@marcustyphoon marcustyphoon reopened this Jun 29, 2023
@marcustyphoon marcustyphoon marked this pull request as draft June 29, 2023 21:32
@marcustyphoon marcustyphoon removed the request for review from AprilSylph July 9, 2023 06:35
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