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
EZP-27572: Markdown specs for the selection buttons. #29
Conversation
6af4559
to
b20d6f5
Compare
doc/spec/selection-button.md
Outdated
``` | ||
|
||
*Constraints:* | ||
input[type='checkbox'] |
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.
?
doc/spec/selection-button.md
Outdated
|
||
|
||
# Markup | ||
|
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.
maybe a short sentence saying the behavior is implemented in a custom element ez-selection-activate-element
doc/spec/selection-button.md
Outdated
|
||
# Markup | ||
|
||
* Here is a simple markup example of a selection button that can be enabled with one or more element and that can be enabled by every checkbox of <ez-selection-activate-element> : |
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.
instead of a list, add sub title, like:
## Enable a button based on a checkboxes
...
## Enable a button only if one element is selected
...
## Enable a button based on a subset of checkboxes
...
b20d6f5
to
6778868
Compare
doc/spec/selection-button.md
Outdated
|
||
* Specification for https://jira.ez.no/browse/EZP-27572 | ||
|
||
Goal here is to provide activable elements (like buttons) that are enabled and can triggers actions only if some elements of a list are selected. |
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 wrap line to something like 100
doc/spec/selection-button.md
Outdated
|
||
# Markup | ||
|
||
## Enable a button based on a checkboxes |
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.
should be: ### Enable a button based on a checkbox
doc/spec/selection-button.md
Outdated
|
||
This behavior is implemented by the `ez-selection-activate-element` custom element. | ||
|
||
# Markup |
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.
should be ## Markup
doc/spec/selection-button.md
Outdated
@@ -0,0 +1,61 @@ | |||
## Provides the suggested markup for the selection buttons |
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.
# Enable button(s) based on a selection
doc/spec/selection-button.md
Outdated
</ez-selection-activate-element> | ||
``` | ||
|
||
## Enable a button only if one element is selected |
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.
### Enable a button only if one element is selected
doc/spec/selection-button.md
Outdated
``` | ||
|
||
|
||
## Enable a button based on a subset of checkboxes |
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.
###
doc/spec/selection-button.md
Outdated
</ez-selection-activate-element> | ||
``` | ||
|
||
*Constraints:* |
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.
### Contraints
doc/spec/selection-button.md
Outdated
*Constraints:* | ||
|
||
* The activable element must have the `ez-js-activable-element` class. | ||
* If the activable element should be enabled only by specific checkboxes or radio button, those must have a class that matches <ez-selection-activate-element>'s `selection-selector`. |
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.
The custom element should be between backticks
6778868
to
a823233
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.
besides heading level +1
doc/spec/selection-button.md
Outdated
activable element should have the `ez-js-activable-element-single-selection` in addition to | ||
`ez-js-activable-element` class. | ||
|
||
# How it works |
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.
##
a823233
to
9678359
Compare
JIRA: https://jira.ez.no/browse/EZP-27572
Suggested markdown for the selection buttons