Skip to content
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

Facets #126

Merged
merged 35 commits into from
Dec 31, 2014
Merged

Facets #126

merged 35 commits into from
Dec 31, 2014

Conversation

jure
Copy link
Contributor

@jure jure commented Dec 23, 2014

No description provided.

@jure jure changed the title Facets WIP: Facets Dec 23, 2014
@jure
Copy link
Contributor Author

jure commented Dec 23, 2014

CrossRef facets:

publisher-name,
funder-name,
type-name

PLOS facets:

journal,
article_type,
publication_date

jure added 18 commits December 23, 2014 22:34
… by moving it to the right. Addresses part of #119.
…meters to SearchController + cleaning of empty parameters submitted due to limitations of GET form requests. Fixes #122.
…onsistency with the PLOS interface. Some more style fixes. Fixes second part of #119.
…licked will reload the page with the facet removed. Clicking on the selected facet link will have the same mechanism. Fixes fourth part of #119
…t to distinguish from primary buttons, fixes part of #118.
…ansition some existing to using either pages (for existing paged functionality) and paginate (which makes pages out of standard array based requests.
jure added a commit that referenced this pull request Dec 31, 2014
@jure jure merged commit 491acb8 into master Dec 31, 2014
@jure jure deleted the facets branch December 31, 2014 12:12
@jure jure changed the title WIP: Facets Facets Jan 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant