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

Register member and required fields - missing asterix (*) #1767

Closed
schlrech opened this issue Jul 7, 2016 · 8 comments
Closed

Register member and required fields - missing asterix (*) #1767

schlrech opened this issue Jul 7, 2016 · 8 comments
Assignees
Labels
type: bug A problem that should not be happening
Milestone

Comments

@schlrech
Copy link

schlrech commented Jul 7, 2016

I missing the star (*) by the fields there are required.

Birthday is for us a required field. After User logged in then popup the usersettings when missing the birthday. there are 20 extendend fields but nothing have a star (*) for the required fields.

in the registersite itself there have stars

@tgtje
Copy link
Contributor

tgtje commented Jul 9, 2016

Correct, Required field for sign up !
Usersettings (suppose change) are using the same (if selected) but not the required > if you demand that field to be a must... talk with user or let him/her sign up again.
So it is not an issue, but a would like to .. (add option/enhancement).

@CaMer0n CaMer0n added the type: enhancement An improvement or new feature request label Jul 9, 2016
@Moc Moc added this to the Future milestone Feb 28, 2018
@Moc Moc changed the title Register member and required fields Register member and required fields - missing asterix (I) Feb 28, 2018
@Moc Moc changed the title Register member and required fields - missing asterix (I) Register member and required fields - missing asterix (*) Feb 28, 2018
@Moc Moc self-assigned this Nov 5, 2019
@Moc Moc added type: bug A problem that should not be happening and removed type: enhancement An improvement or new feature request labels Nov 14, 2019
@Moc Moc modified the milestones: Future, e107 2.3.0 Nov 14, 2019
@Moc
Copy link
Member

Moc commented Nov 14, 2019

@CaMer0n As discussed, missing CSS class for .required

@Moc Moc assigned CaMer0n and unassigned Moc Dec 9, 2019
CaMer0n added a commit that referenced this issue Apr 20, 2020
@CaMer0n
Copy link
Member

CaMer0n commented Apr 20, 2020

@Moc I'm unable to reproduce this. How about you?

@Moc
Copy link
Member

Moc commented Apr 20, 2020

According to my comment 5 months ago, I was able to reproduce it but there was no CSS styling that added the Asterix

@CaMer0n
Copy link
Member

CaMer0n commented Apr 21, 2020

Tested SIGNUP with e107v4a, bootstrap3, kubrick, gaia and freelancer.

image

Gaia and Freelancer (@Jimmi08) are missing the extended user fields in their templates.

CaMer0n added a commit that referenced this issue Apr 21, 2020
@CaMer0n
Copy link
Member

CaMer0n commented Apr 21, 2020

USERSETTINGS tested also without issues.

@CaMer0n
Copy link
Member

CaMer0n commented May 14, 2020

Closing.

@CaMer0n CaMer0n closed this as completed May 14, 2020
@Jimmi08
Copy link
Contributor

Jimmi08 commented Sep 28, 2022

needed css

.required:before {
  content: "*";
  color: red;
  float: right;
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

5 participants