Closed benjaminspak closed 3 years ago
Hi @benjaminspak can i help you with this
@Aarush-Goyal what do you want to improve?
@benjaminspak First of all, there is a typo it should be available jobs. Secondly, the navbar has a better colour contrast so the "components" in the navbar should be a shade of white. Third, there should be some good padding and margin-top margin-bottom between the cards Fourth, There should be an image in the cards which should show the logo of the company Fifth, It would be nice if there is an estimated salary overview Sixth, in the navbar, the justify alignment should be center.
@benjaminspak if you need a sample design I can make it I am a professional UI UX designer
@benjaminspak I have made a PR Added UI Design for Job List #27 in which I have added a png image of the ui which will help you in improvising it.
Also here is a link to the Figma project I have made which will help you with the custom CSS : https://www.figma.com/file/OvJjCK2QacxnsvxfYXAVSq/Job-List?node-id=0%3A1
@benjaminspak pls review the pr and merge it if you feel it is according to the parameters.
Currently pushed as design, will review on new style to implement in project. Merged. Merge request #27 by Aarush-Goyal closed
@igavelyuk is the new style live? (I haven't looked)
If this hasn't been coded yet, @Aarush-Goyal please create an issue: "All Jobs - View - v4" with the image of your new design and basic requirements stating that it needs to be coded in bootstrap and React.
If this has already been coded, please let me know. @igavelyuk
------- V3 -------- Highlight what improvements you would like to make.