Skip to content

Managed search box on inner pages. #142#149

Merged
finnsky merged 6 commits intomasterfrom
search_form_fix
Jul 14, 2018
Merged

Managed search box on inner pages. #142#149
finnsky merged 6 commits intomasterfrom
search_form_fix

Conversation

@finnsky
Copy link
Contributor

@finnsky finnsky commented Jun 20, 2018

what

  • Added inner pages search. Basic theming and functional css.
  • Added some global fixes in header.

references

@finnsky finnsky requested review from goruha and osterman June 20, 2018 14:45
@osterman
Copy link
Member

osterman commented Jun 20, 2018

Thanks! here are a few issues:

1) search results should slide down from search box and flow over the nav bar

image

2) height of nav bar changed considerably

Bad
image
Good
image

3) Rendering on the homepage shows a thin black strip (the shadow box).

I cannot screenshot it because it goes away too quickly, but it doesn't happen on the live site. It's not a pretty page rendering.

image

4) The search results do not expand over/on-top-of the white area.

I discovered a bug after we merged the latest CSS changes. They expand the header section. This was previously working.

Bad
image
Good
image

4) remove borders from input box

They appear after I click the [X].

image

Bad
image

Good
image

5) rendering jumps around

When I click around, the margin/padding above the breadcrumbs seems to be calculated after page load. This changes the feel of the site as loading new pages is not "smooth". This is not happening on live site.
image

Copy link
Member

@osterman osterman left a comment

Choose a reason for hiding this comment

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

This is a good start! There are some UI/CSS issues that I'd like to address before we merge. Also, the search results on the homepage no longer expand over the content, instead they expand the dark gray header.

@finnsky
Copy link
Contributor Author

finnsky commented Jul 12, 2018

Please take a look. I moved slack between breadcrumbs and search. But we need to disquss it.
https://gyazo.com/9dc64caefb208b6aaa250d9bc6b86883

@osterman
Copy link
Member

osterman commented Jul 12, 2018

Thanks @finnsky! Still see some of the same issues. (I did not git pull)

@osterman
Copy link
Member

Looks perfect. Just one nitpick. Searchbox font size should inherit from subnav (13px)

@osterman
Copy link
Member

osterman commented Jul 13, 2018

Can you put the badge like this? Doesn't have to be pixel perfect to this example. We can hide the slack badge on smaller screens.

image

or maybe

image

Copy link
Member

@osterman osterman left a comment

Choose a reason for hiding this comment

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

  • Fix font-size in subnav search box.
  • Change position of slack badge.

Then good to go.

@finnsky finnsky merged commit 9241eac into master Jul 14, 2018
@finnsky finnsky deleted the search_form_fix branch July 14, 2018 07:25
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.

2 participants