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

Rework form.error #11

Open
wants to merge 1 commit into
base: 4.13.x
Choose a base branch
from
Open

Rework form.error #11

wants to merge 1 commit into from

Conversation

zoglo
Copy link
Contributor

@zoglo zoglo commented Oct 27, 2023

Rework form error styles

This PR introduces new styles for forms when an error occured:

  • added box styles
  • does not use position absolute anymore
  • leverage grid properties (using order as every form is a grid in this demo)
  • consider modules (login/registration) -> fixes Error message displacement #9 as well

Screenshots:

Modules

modules

Forms

form

- add box styles
- do not use position absolute
- leverage grid properties
- consider modules (login/registration)
@de-es
Copy link
Contributor

de-es commented Mar 29, 2024

@aschempp Spricht etwas gegen einen Merge? Enhancements und der Bugfix funktionieren. 😁

@aschempp
Copy link
Member

This is up to @sascha-mueller, he's responsible for the frond end (CSS etc.).

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.

None yet

3 participants