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
Improve /companies page #3552
Improve /companies page #3552
Conversation
The cards now employ the `<Card />` component, making them resemble the rest of the webapp more. Also some other minimal styling changes. Finally, the whole card is a link, and not just the image, which has been an annoying bug for a really long time. `CompaniesPage` is rewritten to a functional component. `isHoverable` prop is added for cards that work as links.
|
||
/** Small */ | ||
tight?: boolean; | ||
|
||
/** Shadow */ | ||
shadow?: boolean; | ||
|
||
/** Hidden overflow */ | ||
overflow?: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you need to comment on the prop name, you've chosen a bad name.
I still think we need the option to filter and sort on these companies, but that requires more work and I have an event to attend. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great changes! Much more readable code and will be a lot easier to make further changes 🚀 Also, using the Card component makes it look a lot better already.
Description
The cards now employ the
<Card />
component, making them resemble the rest of the webapp more. Also some other minimal styling changes. Finally, the whole card is a link, and not just the image, which has been an annoying bug for a really long time.CompaniesPage
is rewritten to a functional component.isHoverable
prop is added for cards that work as links.Result
Before
Screen.Recording.2023-02-11.at.14.59.52.mov
After
Screen.Recording.2023-02-11.at.14.58.50.mov
Testing
Tested on light and dark mode, and on mobile (some minimal changes here to the expanded text functionality).
Resolves ABA-270