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

Re-applies fair search design #1936

Merged
merged 1 commit into from
Oct 18, 2016
Merged

Re-applies fair search design #1936

merged 1 commit into from
Oct 18, 2016

Conversation

ashfurrow
Copy link
Contributor

Fixes #1920.

I couldn't quite get the colours right since the icon is a different image – I thought it made more sense for the icons/text size to match the main search than to adhere to exactly what it used to look like. @katarinabatina I'm happy to work on this more, here's what fair search looks like now:

simulator screen shot oct 18 2016 12 12 50 pm

Main search remains unchanged.

@orta
Copy link
Contributor

orta commented Oct 18, 2016

close button feels nicer too 👍

@sarahscott
Copy link
Contributor

Wunderschön! I love the ARSearchViewControllerStyling class 👏

@sarahscott sarahscott merged commit 7977be4 into v3 Oct 18, 2016
@sarahscott sarahscott deleted the ashfurrow-fair-search branch October 18, 2016 19:34
ashfurrow pushed a commit that referenced this pull request Feb 20, 2020
Rename enablePriceTransparency to AROptionsPriceTransparency for consistency
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.

None yet

5 participants