-
Notifications
You must be signed in to change notification settings - Fork 17
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
Short-term SvelteKit rewrite to-do list #85
Comments
|
most sites which have a language picker to begin with have language at the bottom (from what I've seen), plus it's symmetrical with the theme button so I'm not a huge fan of this idea examples: also where would we even put it that doesn't look asymmetrical |
small thing that would be nice — if navigator.language detects that the browser is using you can test this by running console.log(navigator.language); I know it's a small thing but d/m/y and m/d/y are ambiguous formats, the US alone makes up a decent chunk of our traffic, and y/m/d doesn't read very naturally (according to pomi at least, I have no issues with it personally) and yes I hate m/d/y too but this will probably stop a lot of confused users in the future, and shouldn't be terribly hard to implement with some regex |
also another random suggestion just in general can we have posts not have escape characters within the urls if possible, and instead use kebab case (apparently that's the best case style for SEO) https://faithfulpack.net/License%20Update just looks really ugly, add-ons already fix this such as with https://faithfulpack.net/addons/2d-bed-icons and this is even implemented on the beta site, but only for addons I know the post url structure is getting actively rewritten atm because of the weird |
just edit the links, they are not hardcoded |
true but we'd need redirects is it possible to add a redirect field to the new site or something where you can have common misspellings or aliases that will all take you to the same final page |
definitely not gonna do that |
another one you can't open more than one footer category at a time when using the mobile layout, while this seems to be intended behavior I'd definitely rather have it so that you can open as many as you want if people don't like the clutter or whatever we can just have a "close all" button near the top or something if necessary |
EDIT: moved to #86
|
I'd make this a seperate issue because most things you listed will not get implemented in the initial release |
done, renamed this one to a short term TDL to make more sense also til you can edit other people's issue posts this is useful |
|
btw with recent api changes by me and rolf the modding api changes should be much easier to facilitate now since only the api now has to be changed to accommodate new collections rather than figuring out everywhere that's configured to use firestorm it may even be worth moving download JSONs to a firestorm collection with a pack and version key, and adding an interface in the webapp to change that stuff manually, much like the post editor |
We need to figure out a way to exclude Discord-specific FAQ entries from the list, since they're not relevant for the readers of the site |
could just add a field in the faq json to hide it on the site like "discord_exclusive": boolean |
and use it
branding page (docs)very small priority, not needed to finish rewritewindow.navigator.language
)The text was updated successfully, but these errors were encountered: