Comment 11 for bug 1487527

Revision history for this message
Thomas Berezansky (tsbere) wrote :

For the "checkbox as a submit" bit: The "Limit to available items" and "Group Formats and Editions" check boxes do that already, so I was duplicating the functionality those already provide. As mine is literally placed right next to them I saw it as reasonable to duplicate that.

For a couple of other comparative points:

My version does not rely on JavaScript to function. Without JavaScript you have to submit the search yourself, but it still works.
Dan's version seems to act as though you hit the regular search button without JavaScript available/enabled, making me think it should at least be hidden by default and "shown" by JavaScript instead of being there from the start.

My version leaves the search scope at the selected org unit but changes the "depth" of the search. This has effects on some things, such as rendering of Located URIs and copy count displays.
Dan's changes the search scope to the top of the org tree, which will remove the selected library scoping on those features.
Which is better would vary on the goals of the end-user, I suspect, as both methods have pros and cons.