-
Notifications
You must be signed in to change notification settings - Fork 5
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
i3645 finish new numismatics form #3686
Conversation
@@ -120,3 +120,13 @@ | |||
.row .range_limit input.range_begin, .row .range_limit input.range_end { | |||
width: 5em; | |||
} | |||
|
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.
Removing and improving some inline CSS from my previous PR
<div class="search-submit-buttons clearfix col-sm-7"> | ||
<% if sort_fields_select %> | ||
<div class="form-group sort-buttons pull-left"> | ||
<%= content_tag :label, t('blacklight.advanced_search.form.sort_label'), id: 'advanced-search-sort-label', for: 'sort', class: 'control-label' %> | ||
<%= sort_fields_select %> | ||
</div> | ||
<% end %> | ||
<div class="submit-buttons pull-right form-group"> | ||
<%= submit_tag t('blacklight.advanced_search.form.search_btn_html'), class: 'btn btn-primary advanced-search-submit', id: "advanced-search-submit" %> | ||
</div> | ||
|
||
</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.
Adding the two elements that were missing from the form previously
@@ -33,13 +33,31 @@ | |||
allow(view_context).to receive(:facet_limit_for).and_return(nil) | |||
end | |||
|
|||
it "renders fields in the correct order" do | |||
it "renders all expected fields" do |
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 "it" previously didn't accurately describe what was being tested
aa8d403
to
cfc5c35
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.
@sandbergja thank you for your work on the blacklight upgrade! 🍄
When I tested the numismatics form the one bug I can report, but it is not consistent between the fields is that when I tried to edit a search with no results it gave me the '404 something went wrong page'. honeybadger error
020290c
to
e57be3f
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.
Thanks 🍄 ! @sandbergja
@@ -66,6 +66,7 @@ def facets_for_advanced_search_form(solr_p) | |||
# search form, so that the form displays accurate counts for | |||
# them in its dropdowns | |||
advanced_search_facets = blacklight_config.advanced_search.form_solr_parameters['facet.field'] | |||
solr_p[:fq]&.compact! |
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.
Thanks @sandbergja 🍄 !
Closes #3645
To review, make sure that the "View components numismatics" feature is enabled