Skip to content

Implemented Event Card with placeholder images#38

Open
VasilakiG wants to merge 6 commits intodevelopmentfrom
COLOS-6-setup-event-card
Open

Implemented Event Card with placeholder images#38
VasilakiG wants to merge 6 commits intodevelopmentfrom
COLOS-6-setup-event-card

Conversation

@VasilakiG
Copy link
Member

@VasilakiG VasilakiG commented Apr 2, 2024

Issue ID

COLOS-6

What has been done?

  • Made the basic Event Card
  • Placeholder images
  • Actual images in place of the placeholder images
  • Implemented Active Layout around the Event Card

Sceenshots/Videos

screenshot_043

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

languages logos can be part of the database

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but let's include that in a follow-up PR since the database is not ready yet for such things.

display: "flex",
alignItems: "flex-end",
justifyContent: "flex-end",
width: "5%",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove all percentages from everywhere.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed and replaced them with values following the given theme.

height: "100%",
justifyContent: "center",
alignContent: "center",
padding: "1%",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Never use percentages for spacing, it's very unpredictable:

https://mui.com/system/spacing/

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Followed the guide and replaced all the percentages.

borderRadius: "10px",
margin: "auto",
marginTop: "5%",
boxShadow: "0px 0px 1.5px 0px #fff",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made a custom boxShadow inside the theme, under shadows[].

Copy link
Contributor

@vergjor vergjor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also the padding of the elements within the card should be aligned better, compare with the figma design for reference

@VasilakiG VasilakiG force-pushed the COLOS-6-setup-event-card branch from e48b5b4 to 7a707fe Compare October 23, 2024 20:49
…troduced constants for full width and full height
…he title of the competition not wrappable and show dots when overflowed, updated values of elements, so they look closer to the proposed design
@VasilakiG
Copy link
Member Author

VasilakiG commented Oct 24, 2024

also the padding of the elements within the card should be aligned better, compare with the figma design for reference

Aligned them to look similar to the Figma design.

@VasilakiG VasilakiG closed this Oct 24, 2024
@VasilakiG VasilakiG reopened this Oct 24, 2024
@VasilakiG VasilakiG requested review from iboshkov and vergjor October 24, 2024 21:33
@VasilakiG
Copy link
Member Author

Closed the PR because of a small miss-click, wanting to remove an unwanted comment...
Reopened again, sorry

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.

3 participants