-
Notifications
You must be signed in to change notification settings - Fork 289
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
Fixes #8570: Show modal dialog before deletion. #5053
Conversation
@@ -25,8 +25,13 @@ <h3 translate>Packages in {{ repository.name }}</h3> | |||
</div> | |||
|
|||
<div data-block="actions"> | |||
<div bst-modal="removeContent()"> | |||
<div data-block="modal-header" translate>Remove {{ detailsTable.numSelected }} Packages?</div> | |||
<div data-block="modal-body" translate>Are you sure you want to remove the {{ detailsTable.numSelected }} packages selected?</div> |
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 these do we need to use the plural feature that translate gives us? Its annoying to use, but it makes it look much better than seeing '1 packages'
Updated |
LGTM |
@@ -25,8 +25,13 @@ <h3 translate>Packages in {{ repository.name }}</h3> | |||
</div> | |||
|
|||
<div data-block="actions"> | |||
<div bst-modal="removeContent()"> | |||
<div data-block="modal-header" translate-plural>Remove {{ detailsTable.numSelected }} Packages?</div> | |||
<div data-block="modal-body" translate-plural>Are you sure you want to remove the {{ detailsTable.numSelected }} packages selected?</div> |
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.
Is this the right way to use translate-plural? All the other examples in the code are quite a bit more verbose and specific.
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.
No, it's not.
You need to provide translate-n
so it knows whether or not to use the plural and `translate-plural' should have the plural message, see the docs.
@@ -26,8 +26,22 @@ <h3 translate>Docker Images in {{ repository.name }}</h3> | |||
</div> | |||
|
|||
<div data-block="actions"> | |||
<div bst-modal="removeContent()"> | |||
<div data-block="modal-header" | |||
translate-plural |
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.
This extra directive is unneeded right?
ack |
Fixes #8570: Show modal dialog before deletion.
No description provided.