Skip to content

fix(aio): make the search-pane larger (and improve the search-box)#17105

Merged
vicb merged 2 commits into
angular:masterfrom
gkalpak:fix-aio-larger-searchpane-on-mobile
May 30, 2017
Merged

fix(aio): make the search-pane larger (and improve the search-box)#17105
vicb merged 2 commits into
angular:masterfrom
gkalpak:fix-aio-larger-searchpane-on-mobile

Conversation

@gkalpak

@gkalpak gkalpak commented May 30, 2017

Copy link
Copy Markdown
Member

Fixes #17094.

@gkalpak gkalpak added comp: aio action: review The PR is still awaiting reviews from at least one requested reviewer labels May 30, 2017
@mary-poppins

Copy link
Copy Markdown

The angular.io preview for c641025822c0d83010d011418753e82172a5ccfc is available here.

This enables browsers to style it better (e.g. add an `x` button for clearing
the field, which allows users to quickly reset the search query and hide
results).
@gkalpak gkalpak force-pushed the fix-aio-larger-searchpane-on-mobile branch from c641025 to 6f3c4ff Compare May 30, 2017 10:45
@mary-poppins

Copy link
Copy Markdown

The angular.io preview for 6f3c4ff is available here.

@Foxandxss

Copy link
Copy Markdown
Member

I still get a weird behavior

I click it, it gets bigger. I click somewhere else and it gets super long and then goes back to normal.

@gkalpak

gkalpak commented May 30, 2017

Copy link
Copy Markdown
Member Author

This PR does not fix that issue. It just makes the search-pane (where the results are shown) larger and makes the search input field type="search".

@Foxandxss

Copy link
Copy Markdown
Member

Right right, my bad, I had that issue in mind only. My bad.

@petebacondarwin petebacondarwin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@petebacondarwin petebacondarwin changed the title fix(aio): make the search-pane langer (and improve the search-box) fix(aio): make the search-pane larger (and improve the search-box) May 30, 2017
@petebacondarwin petebacondarwin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels May 30, 2017
@petebacondarwin

Copy link
Copy Markdown
Contributor

Mergeable once the Travis flakes sort themselves out.

@gkalpak

gkalpak commented May 30, 2017

Copy link
Copy Markdown
Member Author

The e2e2 Travis failures are "inherited" from master. The rest is green (now).

@vicb vicb merged commit 2f35392 into angular:master May 30, 2017
@gkalpak gkalpak deleted the fix-aio-larger-searchpane-on-mobile branch May 31, 2017 09:57
asnowwolf pushed a commit to asnowwolf/angular that referenced this pull request Aug 11, 2017
…ngular#17105)

* fix(aio): make the search-pane larger

Fixes angular#17094

* feat(aio): give the search-box a type of "search"

This enables browsers to style it better (e.g. add an `x` button for clearing
the field, which allows users to quickly reset the search query and hide
results).
juleskremer pushed a commit to juleskremer/angular that referenced this pull request Aug 28, 2017
…ngular#17105)

* fix(aio): make the search-pane larger

Fixes angular#17094

* feat(aio): give the search-box a type of "search"

This enables browsers to style it better (e.g. add an `x` button for clearing
the field, which allows users to quickly reset the search query and hide
results).
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scroll results pane too short on mobile

6 participants