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

Responsive Web Design Fixes: Global Components and Home #141

Merged
merged 7 commits into from
Dec 8, 2023
Merged

Conversation

willy1989cv
Copy link
Collaborator

Related to: #139

Changes:

Global Sections and Components

  • Header Main Menu : Prevent menu from overriding other components on desktop
  • Language Buttons: prevent line break
  • Next Stop Section:prevent line break
  • ListCard component: responsive behavior

Pages

  • Home
  • Datasets
  • Organizations
  • Topics
  • 101
  • Request Data

@@ -45,146 +45,157 @@ export default function News() {
<p className="mt-3 mb-6 text-center text-base text-gray font-normal">
{t('hm-p-news-text')}
</p>
<div className="container mx-auto flex flex-wrap h-96">

<div className="grid grid-cols-1 md:grid-rows-2 lg:grid-rows-1 lg:grid-cols-3 gap-4">
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would suggest that you add a max-width for this section, otherwise it looks bad on 2k and 4k resolutions

@@ -10,18 +10,22 @@ export default function OpenData101(props) {
}

return (
<div className="relative h-fit">
<div className="relative h-fit mb-[70px] sm:mb-10">
Copy link
Collaborator

Choose a reason for hiding this comment

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

same as for news, I think we need a max-w on this section

@demenech demenech merged commit 5b96b4e into main Dec 8, 2023
1 check failed
@demenech demenech deleted the fix/rwd branch December 8, 2023 11:10
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