-
Notifications
You must be signed in to change notification settings - Fork 12
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
FRONT-2483: Progress pattern. #56
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason the progress isn't indicated, I have to manually refresh, please look into it.
487ebb1
to
1a8700d
Compare
36b3c1c
to
9b34b62
Compare
5983146
to
348e5ae
Compare
348e5ae
to
1a8045d
Compare
bc98af7
to
c7902bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check the place where min and max are now, move them to settings.
message: | ||
type: "text" | ||
label: "Optional description text below the progress bar" | ||
min: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check, min should be moved to settings part.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Min and Max are fields due to free text is not allowed on the settings section and at alexis branch this were also fields, not settings.
type: "text" | ||
label: "Minimum state of prorgress bar" | ||
preview: "0" | ||
max: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for max, should be placed at settings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed. Passing to QA.
…en component progress-bar.html.twig.
a6c0885
No description provided.