Skip to content

Redesign "Home" page - #448

Draft
greenwoodt wants to merge 43 commits into
mainfrom
home/home-page
Draft

greenwoodt wants to merge 43 commits into
mainfrom
home/home-page

Conversation

@greenwoodt

@greenwoodt greenwoodt commented Jul 20, 2026

Copy link
Copy Markdown
Member

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greenwoodt greenwoodt self-assigned this Jul 20, 2026
@netlify

netlify Bot commented Jul 20, 2026

Copy link
Copy Markdown

Deploy Preview for decidim-website ready!

Name Link
🔨 Latest commit c438a16
🔍 Latest deploy log https://app.netlify.com/projects/decidim-website/deploys/6ab134b1a0393b0008b2d9e7
😎 Deploy Preview https://deploy-preview-448--decidim-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@julietapasetti

julietapasetti commented Jul 28, 2026

Copy link
Copy Markdown

First review 'Home':
(I ignored the header)

  • The title and the body have a width of approximately 656px (5 columns).
Captura desde 2026-07-28 14-56-46
  • Change the logos (desktop and mobile)
    Reference OK (size): link
Captura desde 2026-07-28 14-53-50
  • Change the line spacing and check if it is "light."
Captura desde 2026-07-28 14-56-20
  • Change the distance between containers (mobile)
Captura desde 2026-07-28 14-58-01
  • The image extends to the very edges: (mobile)
Captura desde 2026-07-28 14-58-59
  • Change the card's inner margins: (mobile)
Captura desde 2026-07-28 14-59-44

Thanks!

@julietapasetti julietapasetti mentioned this pull request Jul 29, 2026
@elsa-decidim

Copy link
Copy Markdown
Contributor

Hello!

I'm checking the homepage, and in the header it's quite difficult to read the menu, idk if it's a design problem or development, but just checking

Also a question (my bad bc I think it has been already answered): the use cases selection that appear in the home, can we change them? And change for example the japanese case for the brasil one

Besides these topics, for me looks great! Good job! I'll wait for @NilHomedes to check it out before moving it to code review

@NilHomedes NilHomedes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @greenwoodt, great work. thanks!

Some changes that need to be done:

  • Modify the image on "Comment and discuss with your community". See figma to take the new image. (I just changed the text of the comments, the design is the same)
  • Those are not external links. So, they should not have the external link icon.
Image

@greenwoodt

Copy link
Copy Markdown
Member Author

Hi @greenwoodt, great work. thanks!

Some changes that need to be done:

* [ ]  Modify the image on "Comment and discuss with your community". See figma to take the new image. (I just changed the text of the comments, the design is the same)

* [ ]  Those are not external links. So, they should not have the external link icon.
Image

Thanks @NilHomedes. Regarding your second comment what icons should be used instead? Shouldn't that be a comment/question for @julietapasetti?

@NilHomedes

Copy link
Copy Markdown
Member

Thanks @NilHomedes. Regarding your second comment what icons should be used instead? Shouldn't that be a comment/question for @julietapasetti?

Use the internal link icon (an arrow)
imatge

@greenwoodt

Copy link
Copy Markdown
Member Author

Hi @greenwoodt, great work. thanks!

Some changes that need to be done:

* [x]  Modify the image on "Comment and discuss with your community". See figma to take the new image. (I just changed the text of the comments, the design is the same)

* [x]  Those are not external links. So, they should not have the external link icon.

These changes are now applied and ready to review @NilHomedes and @elsa-decidim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Home Page

4 participants