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

Stylin' details for bento (based on -stage 2:12p Friday) #68

Closed
2 of 4 tasks
jvine opened this issue Sep 15, 2017 · 3 comments · Fixed by sul-dlss/SearchWorks#1797
Closed
2 of 4 tasks

Stylin' details for bento (based on -stage 2:12p Friday) #68

jvine opened this issue Sep 15, 2017 · 3 comments · Fixed by sul-dlss/SearchWorks#1797
Assignees

Comments

@jvine
Copy link
Contributor

jvine commented Sep 15, 2017

Not sure how much of this is currently WIP in #15

  • please use the btn-primary styling from SearchWorks for the see-all-results links. The current link doesn't stand out enough. Note: it was intentional design to have the "see-all" button up at the top next to the header. If it's not possible to move it within the time constraints, okay; regardless, it needs to be much more visible.

  • base font size should be 15px. Current 16 so it doesn't match either website nor SW.

  • reduce vertical spacing between the elements of each result. Currently individual results are not holding together visually.

  • add the "Full text" label styling (see SW) to the full text links in the Articles+ section.

@jvine jvine added the ready label Sep 15, 2017
@jvine jvine changed the title Stylin' details for bento Stylin' details for bento (based on -stage 2:12p Friday) Sep 15, 2017
@mejackreed
Copy link
Contributor

First item really has a pre-req of #66

@camillevilla
Copy link
Contributor

camillevilla commented Sep 18, 2017

Some of this is addressed by #75. Remaining items:

@drh-stanford
Copy link

Also the "More search tools" heading is off now:

screen shot 2017-09-20 at 10 08 26 am

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 a pull request may close this issue.

5 participants