Skip to content
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

Dataset page #25

Merged
merged 18 commits into from
Aug 30, 2024
Merged

Dataset page #25

merged 18 commits into from
Aug 30, 2024

Conversation

luccasmmg
Copy link
Contributor

No description provided.

* add partners images and video thumbnails

* increase header z index

* Add Heading Component

* add default rounded rule to button component

* Add Hero, Datasets Sections

- wip: contribute, testimonials, faq, newsletter

* add contribute section

* install accordion ui

* fix footer bg

* Add Faqs Section

* remove unused variables

* add ui-avatars as image domain

* add tabs ui component

* hide contribute section image on sm screens

* add testimonials section

* remove ui-avatar api

* add newsleter

* add toaster

* add badge component

* add border right to latest datasets
Copy link

vercel bot commented Aug 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tdc-data-portal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 30, 2024 9:38pm

@luccasmmg
Copy link
Contributor Author

Different icons here: image image image image

The buttons sizes differ from the mockup and the right button is misaligned with the one on the left.

image image

No responsivity in the tabs sections generating horizontal scroll: image

Copy to clipboard button bg color, icon, and spacing differ from the mockup:

image image

The resources icons are a little flat compared with the mockup and the downloads icons differ from the mockup as well: image image

image image

The icon stuff im not really planning on fixing right now, i wanted to see if they had an icon pack first, cause loading the icons as svg images is not really nice since it requires extra http requests just to get the icon (Instead of having it in the mockup from the start) You can check that by using your browser to simulate a slow 3g and opening up the "Downloads" tab, the icons there were taken directly from the mockup, you will see them taking a while to load

The

  • Lack of responsivness i will fix it
  • Missaligned buttons i will fix it
  • The color in the copy to clipboard button as well

@luccasmmg
Copy link
Contributor Author

The problem with taking the icons as images from the figma

Screencast.from.2024-08-30.17-52-27.webm

Copy link
Contributor

@Gutts-n Gutts-n left a comment

Choose a reason for hiding this comment

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

Different icons here:
image
image
image
image

The buttons sizes differ from the mockup and the right button is misaligned with the one on the left.

image
image

No responsivity in the tabs sections generating horizontal scroll:
image

Copy to clipboard button bg color, icon, and spacing differ from the mockup:

image
image

The resources icons are a little flat compared with the mockup and the downloads icons differ from the mockup as well:
image
image

image
image

@Gutts-n
Copy link
Contributor

Gutts-n commented Aug 30, 2024

LGTM.

@luccasmmg luccasmmg merged commit 7fc6581 into main Aug 30, 2024
3 checks passed
@demenech demenech deleted the dataset-page branch September 3, 2024 00:47
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.

2 participants