Skip to content

Fix white bottom on proposal show with voting#14285

Merged
andreslucena merged 7 commits intodevelopfrom
fix/proposals-margin
Mar 18, 2025
Merged

Fix white bottom on proposal show with voting#14285
andreslucena merged 7 commits intodevelopfrom
fix/proposals-margin

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu commented Mar 13, 2025

🎩 What? Why?

Back in #13948, @andreslucena fixed the sticky buttons, on mobile. Unfortunately we have not seen that in the process, we add a white background on proposals show page.

📌 Related Issues

Link your PR to an issue

Testing

  1. Visit any proposal page having votes enabled
  2. Scroll to bottom
  3. See a 140px+ margin bottom
  4. Go to mobile view & see the space is covered by a sticky button
  5. Apply patch
  6. repeat 1+2
  7. See the margin bottom is gone in desktop view
  8. Repeat step 4

📷 Screenshots

Images from: https://nightly.decidim.org/processes/charkarmob/f/5/proposals/8

image
image

♥️ Thank you!

@alecslupu alecslupu added the type: fix PRs that implement a fix for a bug label Mar 13, 2025
github-actions[bot]
github-actions bot previously approved these changes Mar 13, 2025
@alecslupu alecslupu added release: v0.30 Issues or PRs that need to be tackled for v0.30 and removed module: core labels Mar 13, 2025
@alecslupu alecslupu requested review from a team and greenwoodt March 13, 2025 07:09
@alecslupu alecslupu moved this to 👀 In review in Maintainers' backlog Mar 13, 2025
greenwoodt
greenwoodt previously approved these changes Mar 13, 2025
Copy link
Copy Markdown
Member

@greenwoodt greenwoodt left a comment

Choose a reason for hiding this comment

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

Works as expected

@alecslupu alecslupu assigned andreslucena and unassigned greenwoodt Mar 13, 2025
@andreslucena andreslucena changed the title Fix white bottom on proposal show Fix white bottom on proposal show with voting Mar 14, 2025
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

I can confirm this bug is fixed.

I found another one related to this script: if you change the screen size from sm to xl, the space it still there. This seems a weird scenario but I think it can happen in mobile devices (by changing the orientation from vertical to landscape for instance)

I think it's because we aren't recalculating on resize.

Do you want to do it here or should I create a new issue for that one?

@alecslupu
Copy link
Copy Markdown
Contributor Author

I can confirm this bug is fixed.

I found another one related to this script: if you change the screen size from sm to xl, the space it still there. This seems a weird scenario but I think it can happen in mobile devices (by changing the orientation from vertical to landscape for instance)

I think it's because we aren't recalculating on resize.

Do you want to do it here or should I create a new issue for that one?

Let's do it here.

github-actions[bot]
github-actions bot previously approved these changes Mar 14, 2025
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

Stiill happening even with the resize:

Screenshot of the bug

@alecslupu
Copy link
Copy Markdown
Contributor Author

Stiill happening even with the resize:

I actually tested on an Ipad, and i have not seen the issue after implementing the resize. I had some Caching issues though ...

@alecslupu
Copy link
Copy Markdown
Contributor Author

I actually tested on an Ipad, and i have not seen the issue after implementing the resize. I had some Caching issues though ...

@andreslucena this is ready for another round...

Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

👏🏽 👏🏽

Now it works just as I'd expect, thanks!

@andreslucena andreslucena merged commit 2c0337d into develop Mar 18, 2025
118 checks passed
@andreslucena andreslucena deleted the fix/proposals-margin branch March 18, 2025 07:56
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Maintainers' backlog Mar 18, 2025
@alecslupu alecslupu added the no-backport Pull Requests that should not be backported label Mar 18, 2025
@alecslupu
Copy link
Copy Markdown
Contributor Author

Add no-backport, as the feature does not exist in 0.29

entantoencuanto added a commit that referenced this pull request Mar 20, 2025
* feature/signature_workflows:
  Add documentation for signature handler encryption secret and rename environment variable
  Rename method
  Fix title on spec for the 'Create new proposal' page (#14347)
  Fix WCAG search filters navigation (#14014)
  Fix comment cell to not show reply button when passing MAX_DEPTH (#14291)
  Fix editing comment with max length setting (#14275)
  Add title tag on pages for blogs, debates and meetings (#14110)
  Fix white bottom on proposal show with voting (#14285)
  New Crowdin updates (#14318)
  New Crowdin updates (#14217)
  Fix image missing error (#14175)
  Add delay of `spam_analysis` queue (#14304)
  Fix hide comment from interface (#14301)
  Change "Valuator" for "Evaluator" (#13684)
  Fix QR code generation (#14300)
  Patch backporter action (#14292)
  Change `hide_voting` argument on proposal cell to `show_voting` (#14267)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: core no-backport Pull Requests that should not be backported 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
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Footer and header extra margin

3 participants