Skip to content
This repository has been archived by the owner on Jun 18, 2024. It is now read-only.

CommandBar Search item doesn't work in mobile / narrow pages #102

Closed
ilyalehrman opened this issue Sep 8, 2015 · 6 comments
Closed

CommandBar Search item doesn't work in mobile / narrow pages #102

ilyalehrman opened this issue Sep 8, 2015 · 6 comments

Comments

@ilyalehrman
Copy link

The CommandBar component's Search item only seems to work on large displays with enough real estate to show the search item text box. When collapsed, the search item doesn't do anything.

@wdo3650
Copy link
Contributor

wdo3650 commented Sep 9, 2015

Thanks for creating this issue. Does the search box fail to expand when the magnifying glass icon is selected or does the search box not do anything after text is entered?

@ilyalehrman
Copy link
Author

The search box fails to expand when you click on the magnifying glass.


From: wdo3650 notifications@github.com
Sent: Tuesday, September 8, 2015 8:30 PM
To: OfficeDev/Office-UI-Fabric
Cc: ilya lehrman
Subject: Re: [Office-UI-Fabric] CommandBar Search item doesn't work in mobile / narrow pages (#102)

Thanks for creating this issue. Does the search box fail to expand when the magnifying glass icon is selected or does the search box not do anything after text is entered?

Reply to this email directly or view it on GitHubhttps://github.com//issues/102#issuecomment-138739036.

@wdo3650
Copy link
Contributor

wdo3650 commented Sep 9, 2015

Thanks for the clarification. Which browsers and OS's are you seeing this in?

@ilyalehrman
Copy link
Author

Windows 10: IE 11/ Edge / FF 40.0.4 / Chrome 45.0.2454.85 m.

Furthermore, after re-expanding the page, --mainArea CommandItems do not come back into view (with the occasional exception of the first CommandItem. If this isn't related I can log it in a separate issue...

@wdo3650
Copy link
Contributor

wdo3650 commented Sep 9, 2015

Thanks for letting us know about this other issue. We've resolved both of these issues in the develop branch and are in the process of merging these fixes into master.

@ilyalehrman
Copy link
Author

Good deal! Looking forward to the update!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants