-
-
Notifications
You must be signed in to change notification settings - Fork 836
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
Design nicer error/placeholder pages #596
Comments
404 pages should contain a login form. |
Kind of also the same as #252 but meh |
I guess a temporary workaround can be to have a refresh to the login page in the 403 and 404.html pages ... That's how I fixed it for the time being anyway ... |
@edd-- Yep, that's a good idea! |
i have a simple idea for error pages: Should I open a PR to add these @tobscure @franzliedke ? |
I'd prefer to use the compiled (if possible) stylesheet and use that styling @TristianK3604 . |
@luceos Why? I think it's good when error pages are self-contained, and so far that's what we have been doing... |
I wasn't aware about that decision. I nevertheless don't think it's a big deal to include the stylesheet, if it doesn't exist it won't break the page anyway. |
The design aspect of this issue is already solved in 9392e1b and b7c1cc5 (See https://discuss.flarum.org/asdf for an example) |
This comment has been minimized.
This comment has been minimized.
The user mentions, posts, and discussions pages now all have empty messages. That leaves "too few characters = no discussions" as the only issue; as such, I will reopen https://github.com/flarum/core/issues/773 (if it's not really needed, stable bot will remove it) and close this. |
Remaining from #118:
The text was updated successfully, but these errors were encountered: