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

Missing Asterix after Extended User field introduced #3676

Open
wrxxy opened this issue Feb 14, 2019 · 9 comments
Open

Missing Asterix after Extended User field introduced #3676

wrxxy opened this issue Feb 14, 2019 · 9 comments
Labels
type: bug A problem that should not be happening
Milestone

Comments

@wrxxy
Copy link

wrxxy commented Feb 14, 2019

Ok when signing up to website with mandatory fields the asterix is shown as pic below,
image

but when the Extended User fields have been modified after a user has signed up the asterix are not shown as the pic below shows
image

Just wondering if this issue could be address please thx.....

@CaMer0n CaMer0n added the type: bug A problem that should not be happening label Feb 14, 2019
@CaMer0n CaMer0n added this to the e107 2.2.0 milestone Feb 14, 2019
@wrxxy
Copy link
Author

wrxxy commented Mar 1, 2019

Hi Cameron just letting you know that this issue is still current.

@CaMer0n CaMer0n reopened this Mar 2, 2019
@Moc
Copy link
Member

Moc commented Mar 11, 2019

@wrxxy What's the status on this with the latest files?

@Moc Moc added the status: awaiting feedback This issue may be fixed and is awaiting the original poster to confirm the fix. label Mar 11, 2019
@wrxxy
Copy link
Author

wrxxy commented Mar 11, 2019

@Moc just checked and this issue is still current , on sign up its fine but when your updating your user settings it doesn't show asterix's as per title...

@Moc Moc removed the status: awaiting feedback This issue may be fixed and is awaiting the original poster to confirm the fix. label Mar 13, 2019
@Moc
Copy link
Member

Moc commented Apr 29, 2019

Does your theme use a custom usersettings_template.php? If yes, look for {FIELDNAME} and make sure to add {REQUIRED} if it's not there: {FIELDNAME} {REQUIRED}

@Moc Moc added the status: awaiting feedback This issue may be fixed and is awaiting the original poster to confirm the fix. label Apr 29, 2019
@Moc Moc modified the milestones: e107 2.2.0, Future Apr 29, 2019
@Moc
Copy link
Member

Moc commented Oct 29, 2019

@wrxxy Any update on this? :)

@wrxxy wrxxy closed this as completed Nov 5, 2019
@Moc Moc reopened this Nov 14, 2019
@Moc Moc removed the status: awaiting feedback This issue may be fixed and is awaiting the original poster to confirm the fix. label 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

@CaMer0n
Copy link
Member

CaMer0n commented Apr 29, 2020

@Moc Closing. But please re-open if someone can confirm the issue still exists.

@CaMer0n CaMer0n closed this as completed Apr 29, 2020
@Jimmi08
Copy link
Contributor

Jimmi08 commented Jul 6, 2023

Just note for needed css

 span.required:after {
     content: "*";
     color: red;
     font-size: 24px;
     margin-left: 4px;
 }

After apply there is asterisk 2x with force update
$text .= "<div class='fborder' style='text-align:center'><br />" . str_replace("*", "<span class='required'>*</span>", LAN_USET_9) . "<br />" . LAN_USET_10 . "<br /><br /></div>";

@Jimmi08
Copy link
Contributor

Jimmi08 commented Feb 1, 2024

Still needed theme fix, I am glad that I wrote it here :)

@Moc Moc reopened this Feb 1, 2024
@Moc Moc modified the milestones: e107 2.3.0, e107 2.4.0 Feb 1, 2024
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

4 participants