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

Bump decidim from 0.27.3 to 0.28.2 #42

Open
wants to merge 46 commits into
base: main
Choose a base branch
from
Open

Bump decidim from 0.27.3 to 0.28.2 #42

wants to merge 46 commits into from

Conversation

HHassig
Copy link
Contributor

@HHassig HHassig commented Sep 24, 2024

No description provided.

@HHassig HHassig changed the title update awesome spec to copy decidim-ub and fix alternative landing Bump decidim 0.27.3 to 0.28.2 Sep 24, 2024
@HHassig HHassig requested a review from fblupi September 24, 2024 17:05
@HHassig
Copy link
Contributor Author

HHassig commented Sep 24, 2024

@fblupi, regarding the awesome specs, i have simply copied the spec file from decidim-ub

@fblupi fblupi changed the title Bump decidim 0.27.3 to 0.28.2 Bump decidim from 0.27.3 to 0.28.2 Sep 27, 2024
Copy link
Member

@fblupi fblupi left a comment

Choose a reason for hiding this comment

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

I checked the styles and I see some weird ones:

Search bar icon color:

Screenshot 2024-09-27 at 10 21 39

Links to "see all" in content blocks:

Screenshot 2024-09-27 at 10 23 27

Day of the month color in the meetings content block:

Screenshot 2024-09-27 at 10 24 00

Language selector may look better with black color instead of white.

Screenshot 2024-09-27 at 10 24 53

Section separator on main menu may look better being transparent:

Screenshot 2024-09-27 at 10 26 42

When you set colours to the organization, title decorators looks weird:

Screenshot 2024-09-27 at 10 30 56

Blue border color in buttons should be white, black or transparent to look better (I see them on modals and other pages, just navigate a little bit):

Screenshot 2024-09-27 at 10 30 11

app/packs/stylesheets/decidim/decidim_application.scss Outdated Show resolved Hide resolved
db/schema.rb Outdated Show resolved Hide resolved
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