Skip to content

Fix UX mobile budget page adjustements#14028

Merged
alecslupu merged 8 commits intodecidim:developfrom
octree-gva:fix/ux_mobile_budget_page
Feb 18, 2025
Merged

Fix UX mobile budget page adjustements#14028
alecslupu merged 8 commits intodecidim:developfrom
octree-gva:fix/ux_mobile_budget_page

Conversation

@rsdeus
Copy link
Copy Markdown
Contributor

@rsdeus rsdeus commented Feb 5, 2025

🎩 What? Why?

On Mobile, on a Budget page:

  • The layout of this page is not optimal on Mobile
  • The dropdown Trier les projets par is on two-line
  • The buttons Ajouter don't have the right horizontal padding

📌 Related Issues

#14027

📷 Screenshots

image

♥️ Thank you!

@github-actions github-actions bot added module: core module: budgets type: fix PRs that implement a fix for a bug labels Feb 5, 2025
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: ['type: feature', 'type: change', 'type: fix', 'type: removal', 'target: developer-experience', 'type: internal']

@andreslucena andreslucena changed the title Fix ux mobile budget page adjustements Fix UX mobile budget page adjustements Feb 6, 2025
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

LGTM

Before:
Screenshot from 2025-02-18 12-13-23

After:
image

@alecslupu alecslupu merged commit 3ba85f5 into decidim:develop Feb 18, 2025
@alecslupu alecslupu added the release: v0.29 Issues or PRs that need to be tackled for v0.29 label Feb 18, 2025
@alecslupu alecslupu added the release: v0.28 Issues or PRs that need to be tackled for v0.28 label Feb 18, 2025
@andreslucena
Copy link
Copy Markdown
Member

(Note for maintainers)

FYI this is out of the v0.30.0.rc1, but we can add it so it is in rc2 (or v0.30.1, whatever comes first)

@alecslupu alecslupu added the release: v0.30 Issues or PRs that need to be tackled for v0.30 label Feb 18, 2025
entantoencuanto added a commit that referenced this pull request Feb 18, 2025
* develop:
  Change release patch instructions in Releases internal documentation (#14061)
  Suggested hashtags in Proposals (#14103)
  Make transferable authorization when duplicate user is ephemeral (#13981)
  Fix UX mobile budget page adjustements (#14028)
  Fix develop pipeline (#14104)
  New Crowdin updates (#14106)
  New Crowdin updates (#14100)
  New translations en.yml (Catalan) (#14098)
  New Crowdin updates (#14093)
  New translations en.yml (Basque) (#14090)
  New Crowdin updates (#14081)
@alecslupu alecslupu mentioned this pull request Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: budgets module: core release: v0.28 Issues or PRs that need to be tackled for v0.28 release: v0.29 Issues or PRs that need to be tackled for v0.29 release: v0.30 Issues or PRs that need to be tackled for v0.30 type: fix PRs that implement a fix for a bug

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants