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

Add CSS class `xhsRequired` for all required customer data labels #157

Merged
merged 2 commits into from Aug 2, 2017

Conversation

@cmb69
Copy link
Member

cmb69 commented Aug 2, 2017

This way its easy to style these labels, for instance, by appending an
asterisk.

This way its easy to style these labels, for instance, by appending an
asterisk.
@frase-git

This comment has been minimized.

Copy link
Collaborator

frase-git commented Aug 2, 2017

Lässt du das für jeden User offen, oder folgt noch die Klasse im CSS?
Ich wäre für offenlassen, aber die Klasse ohne Regel ins CSS einfügen.
Das ist Komfort und lässt sich gut beschreiben.

@cmb69

This comment has been minimized.

Copy link
Member Author

cmb69 commented Aug 2, 2017

Ich wäre für offenlassen, aber die Klasse ohne Regel ins CSS einfügen.

Okay, gemacht. Was ist mit .xhsFormLabelInl? Die Klasse wird wohl nie verwendet. Weg damit?

@frase-git

This comment has been minimized.

Copy link
Collaborator

frase-git commented Aug 2, 2017

Weg damit? Gerne. Löschen ist immer gut.
Ich weiß jetzt aber wirklich nicht mehr, ob das noch gebraucht wird.
In der Entwicklungsphase brauchte ich das mal unbedingt - es hat sich aber viel geändert ...

@cmb69

This comment has been minimized.

Copy link
Member Author

cmb69 commented Aug 2, 2017

Ich weiß jetzt aber wirklich nicht mehr, ob das noch gebraucht wird.
In der Entwicklungsphase brauchte ich das mal unbedingt - es hat sich aber viel geändert ...

Es würde wohl nur gebraucht, wenn jemand das Template anpasst. Der kann dann aber auch gleich eine andere Klasse vergeben.

cmb69 added a commit that referenced this pull request Aug 2, 2017
@frase-git

This comment has been minimized.

Copy link
Collaborator

frase-git commented Aug 2, 2017

Conflicting files
css/stylesheet.css

@cmb69

This comment has been minimized.

Copy link
Member Author

cmb69 commented Aug 2, 2017

Conflicting files

Ja, ich weiß. Das kann ich beim Mergen korrigieren. Soll ich mergen?

@frase-git

This comment has been minimized.

Copy link
Collaborator

frase-git commented Aug 2, 2017

Jo.

@cmb69 cmb69 merged commit 29282fc into master Aug 2, 2017
@cmb69 cmb69 deleted the required branch Aug 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.