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.

Sign up for GitHub

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

Jump to bottom

calling setLabel(String label) removes required indicator #860

Closed
doctorj opened this issue Oct 20, 2023 · 0 comments
Closed

calling setLabel(String label) removes required indicator #860

doctorj opened this issue Oct 20, 2023 · 0 comments
Assignees
@vegegoku
Labels
bug Something isn't working version 2.x.x Version 2.x.x issues
Projects
Domino UI
Milestone
2.0.0-RC4

Comments

@doctorj
Copy link

doctorj commented Oct 20, 2023

I guess this stand for any AbstractFormElement that is set as required.
Since required indicator is represented as <span> element that is part of labels content, it is lost when setLabel(String label) is called, because entire label content is replaced.
@vegegoku vegegoku self-assigned this Oct 21, 2023
@vegegoku vegegoku added bug Something isn't working version 2.x.x Version 2.x.x issues labels Oct 21, 2023
@vegegoku vegegoku added this to To do in Domino UI via automation Oct 21, 2023
@vegegoku vegegoku added this to the 2.0.0-RC4 milestone Oct 21, 2023
@vegegoku vegegoku changed the title v2: calling setLabel(String label) removes required indicator calling setLabel(String label) removes required indicator Oct 26, 2023
vegegoku added a commit that referenced this issue Oct 26, 2023
@vegegoku
fix #860 calling setLabel(String label) removes required indicator
a34f0eb
Domino UI automation moved this from To do to Done Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vegegoku vegegoku

Labels
bug Something isn't working version 2.x.x Version 2.x.x issues
Projects
Domino UI
  
Done
Milestone
2.0.0-RC4
Development

No branches or pull requests
2 participants
@doctorj @vegegoku