Hi Belen,
Here is another review request. This is to add (in the filter pop-up dialog)
the message "You can only apply one filter to the table. This filter will
override the current filter" when a different filter is currently applied, as
per your design document "Filtering the 'All builds' table" page 7.
The review branch is here: dreyna/second_filter_5960
Notes:
1. Currently the warning string is right-justified, but your design document
has it left-justified next to the button. Is that ok? Otherwise, can you tell
me how to adjust the CSS for "modal-footer"?
2. I ended up with a nested IF for the test to show the message , because I
could not get Django to accept the two tests in one IF.
The test is (a) if there is an existing filter string in the session, and (b)
the existing filter has a different prefix from this dialog's filter prefix,
then show the warning string.
Here is the code in "filtersnippet.html":
<div class="modal-footer">
<button type="submit" class="btn btn-primary">Apply</button>
{% if request.GET.filter %}
{% if request.GET.filter|string_remove_regex:':.*' !=
f.options.0.1|string_remove_regex:':.*' %}
You can only apply one filter to the table. This filter will
override the current filter.
{% endif %}
{% endif %}
</div>
- David
--
_______________________________________________
toaster mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/toaster
