Skip to content
This repository has been archived by the owner on Jan 31, 2018. It is now read-only.

[bug 949000] Switch product list to dropdown #197

Closed
wants to merge 2 commits into from
Closed

[bug 949000] Switch product list to dropdown #197

wants to merge 2 commits into from

Conversation

willkg
Copy link
Member

@willkg willkg commented Dec 12, 2013

The occurrences report now uses a dropdown list of all available
products.

r?

The occurrences report now uses a dropdown list of all available
products.
@willkg
Copy link
Member Author

willkg commented Dec 12, 2013

To test, go to the occurrences report and notice it's now a dropdown.

<input id="id_product" type="text" name="product" value="{{ form.product.value() }}" />
<select id="id_product" name="product">
{% for prod in products %}
<option value="{{ prod }}">{{ prod }}</option>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is missing one thing. You should set the selected attribute on the product you just used for the report.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, whoops!

@willkg
Copy link
Member Author

willkg commented Dec 12, 2013

^^^ Fixes the selected thing.

@rlr
Copy link
Contributor

rlr commented Dec 12, 2013

WFM R+!

@willkg
Copy link
Member Author

willkg commented Dec 12, 2013

Landed in e951be8 [bug 949000] Switch product list to dropdown

@willkg willkg closed this Dec 12, 2013
@willkg willkg deleted the product-list-949000 branch December 12, 2013 23:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants