Homepage redesign redux (updated) #1113

Open
wants to merge 49 commits into
from

Projects

None yet

5 participants

@CloCkWeRX
Contributor
CloCkWeRX commented Dec 2, 2016 edited

Replaces #697

From the original:

Fixes #628

Replaces #674

@Skud :
> I think after you select something from that search UI it should take you direct to the crop page in question (eg. /crops/eggplant) not to search results (unless there's not an exact match).

This should work in production where we're using Elasticsearch but it doesn't seem to work in development. See inline note.

@Skud :

Why is the crop search not an autosuggest? If I type "pea" or "berry" for instance I don't see the list.

Good call. I need to refactor autosuggest a bit. Should I do that now or break into a new story?

Question: Should the signed in user homepage redesign be part of this PR?

Screenshot (updated March 1, 2015)
screen shot 2015-03-01 at 5 07 09 pm

Updated screenshot of homepage:
image
image

tygriffin and others added some commits Jan 13, 2015
@tygriffin tygriffin schema updated a0a14a9
@tygriffin tygriffin sketch basic outline of redesign 53dbc92
@tygriffin tygriffin style search crops area with high contrast font against full-width image d61e33d
@tygriffin tygriffin style map area 4907f9f
@tygriffin tygriffin Merge branch 'dev' of https://github.com/Growstuff/growstuff into hom…
…epage-redesign
c4aa016
@tygriffin tygriffin add a few representative crops under search crops fd0644a
@tygriffin tygriffin configure places map to work on homepage 87610b0
@tygriffin tygriffin move customs styles in overrides 5b4edcd
@tygriffin tygriffin wire up find growers to place search functionality 7eb95d6
@tygriffin tygriffin get h1 title from ENV variable 33108ae
@tygriffin tygriffin uncomment other parts of page 33add70
@tygriffin tygriffin wire up find crops to search crops functionality 2833b1d
@tygriffin tygriffin revert to previous bootstrap non-fullwidth container dff41ee
@tygriffin tygriffin increase legibility of label with text shadow 905f36f
@tygriffin tygriffin target header sign up link because we removed the other one a79d96c
@tygriffin tygriffin extract strings out of new homepage design into locale file 2d45419
@tygriffin tygriffin added full-width layout for home ad39212
@tygriffin tygriffin add image behind title 15ea04d
@tygriffin tygriffin center map 9ecf2b0
@tygriffin tygriffin simplify layout by removing interesting crops 0c3043c
@tygriffin tygriffin set map view automatically when typing in address place name on homepage c423cc1
@tygriffin tygriffin add image attribution f457a2f
@tygriffin tygriffin crops search goes direct to crop detail page if there's an exact match 8abb95a
@tygriffin tygriffin move everything not part of the top banner into a container 0914853
@tygriffin tygriffin increase title size ff6864a
@tygriffin tygriffin add back interesting crops 67d1b76
@tygriffin tygriffin fix locale spec bde7565
@tygriffin tygriffin add contract screen on map 7529bdc
@tygriffin tygriffin use negative margin for positioning f34469c
@tygriffin tygriffin refactor styles 05f8d4e
@tygriffin tygriffin fix locale inconsistency 4146ace
@tygriffin tygriffin get rid of old layout we're not using b62e15f
@tygriffin tygriffin map zooms only on find growers submit 0ffbcd4
@tygriffin tygriffin remove unnecessary locale strings 5067e6c
@tygriffin tygriffin removed extraneous tests c1ab1f6
@tygriffin tygriffin catch up with dev 6dc37e2
@tygriffin tygriffin wip: exact search 01ab9ec
@tygriffin tygriffin go to direct to crop detail if exact match d23c21e
@tygriffin tygriffin catch up with dev e881d6e
@tygriffin tygriffin add back exact search functionality 734228b
@tygriffin tygriffin put find growers input box flush against map border cf042f1
@CloCkWeRX CloCkWeRX Merge remote-tracking branch 'upstream/dev' into HEAD 9ca9637
@CloCkWeRX CloCkWeRX Fix missed partial 8e663de
@CloCkWeRX CloCkWeRX Revert changes from 2872a1c a2bba6f
@CloCkWeRX CloCkWeRX Require app constants 13dd20c
@CloCkWeRX CloCkWeRX Update schema ef884cb
@CloCkWeRX CloCkWeRX Merge CSS from less to sass a bit better
b217cbe
@CloCkWeRX
Contributor

So... this is a bit more mergable, but I think its a bit less nice that the current logged in user homepage.

It might be better to redo this to mix in the map feature to the existing content or otherwise split into 'logged in experience' vs 'new user journey'

@CloCkWeRX CloCkWeRX Remove dead code
b9488a6
@coveralls

Coverage Status

Coverage decreased (-1.02%) to 86.655% when pulling b9488a6 on CloCkWeRX:homepage-redesign-redux into 266455e on Growstuff:dev.

@coveralls

Coverage Status

Coverage decreased (-1.02%) to 86.655% when pulling b9488a6 on CloCkWeRX:homepage-redesign-redux into 266455e on Growstuff:dev.

@pozorvlak
Member

Lots of Missing template home/_blurb errors in CI: did you forget to add a file?

@CloCkWeRX
Contributor

The merge was quite a pain, it was probably eaten at some point. Will restore in a bit

@Br3nda Br3nda Merge branch 'dev' into homepage-redesign-redux
27f3898
@coveralls
coveralls commented Dec 15, 2016 edited

Coverage Status

Coverage decreased (-1.1%) to 87.251% when pulling 27f3898 on CloCkWeRX:homepage-redesign-redux into 148dfa7 on Growstuff:dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment