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.

Sign up for GitHub

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

Jump to bottom

Navbar search box is broken on mobile device #159

Open
amjithps opened this issue Jan 22, 2018 · 4 comments
Open

Navbar search box is broken on mobile device #159

amjithps opened this issue Jan 22, 2018 · 4 comments

Comments

@amjithps
Copy link

Navbar search box is broken in mobile devices.
image
@codefactorymx
Copy link
Contributor

Just add display: Inline-flex to the parent div of the search text input. I'll add a pull request with the fix

@codefactorymx
Copy link
Contributor

Add the following style to src/scss/win/_main.scss

.navbar-right {
display: inline-flex
}

@amjithps
Copy link
Author

I am using custom build application and using the CSS /css/winstrap.css

Should i want to add this style "display: inline-flex" to all navbar-right media's?

@codefactorymx
Copy link
Contributor

Hi Amjith, a fix was sent to MS guys. Meanwhile, you can download the css from here
https://github.com/codefactorymx/winstrap/tree/master/dist/css

mlijanto pushed a commit that referenced this issue Mar 27, 2018
@codefactorymx @mlijanto
Fix for issues #115 and #159 - Navbar search box and progress ring/bar (
1e96629 
#162)

* Added inline-flex for parent of search bar, fix issues in mobile

* Issue # 115 - Progress Bar and Progress Ring
Fixed the progress bar and ring issues, added the center alignment option.
Documentation was updated

* Fixed the documentation of NavBar, Hamburguer icon is not displayed.
Container is not centering the content

* Fixed the documentation of NavBar, changed the overrides to correct file.

* Removed override from incorrect file

* Recompiled optional css

* Fixed documentation of NavBar, chevron icons in blue bar.
codefactorymx added a commit to codefactorymx/winstrap that referenced this issue May 4, 2018
@codefactorymx
Merge pull request #1 from winjs/master
ce8e6d7 
Fix for issues winjs#115 and winjs#159 - Navbar search box and progress ring/ba…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@amjithps @codefactorymx