Skip to content
This repository has been archived by the owner on Feb 28, 2022. It is now read-only.

Search and sort bar repositioning (341+342) #373

Closed
wants to merge 8 commits into from
Closed

Search and sort bar repositioning (341+342) #373

wants to merge 8 commits into from

Conversation

NaginiChen
Copy link
Collaborator

Description

The current design on Marvel has the search bar centered and the sort bar to the left. These have been moved to match the design. Much of the code in AdminPage.js, CardGrid.js, and CategoryList.js was rearranged to allow the sorting by distance functionality to still work.

NOTE: There is one small thing with the UI. The sort bar is a little bit squished against the category section on laptops and the header on mobile. I'm aware that the CSS is being changed/refactored right now, but I'm not sure in what way. Someone else should add some margin to the sort bar.

UIchange

Fixes #

This is for issues #341 and #342.

Motivation and Context

This is a UI enhancement and is the first steps to making the admin page look like the mock-up design on Marvel (https://marvelapp.com/c054d54/screen/57793613).

Type of change

  • UI Change Only

@RajChaudhry RajChaudhry self-requested a review July 29, 2019 13:48
@RajChaudhry
Copy link
Collaborator

Hey Brandon! There is still some functionality problems with this change. Specifically, upon checking categories or writing in the search box, no resources are filtered.

@RajChaudhry
Copy link
Collaborator

Closing due to big changes via css refactor #372

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

Successfully merging this pull request may close these issues.

2 participants