Skip to content

Comments

Composer quote improvements#35835

Merged
ClearlyClaire merged 16 commits intomainfrom
feat/quote-post-composer-fixes
Aug 21, 2025
Merged

Composer quote improvements#35835
ClearlyClaire merged 16 commits intomainfrom
feat/quote-post-composer-fixes

Conversation

@ChaosExAnima
Copy link
Contributor

@ChaosExAnima ChaosExAnima commented Aug 20, 2025

Fixes MAS-538, MAS-528, and MAS-539.

This PR connects the new modal created in #35762 to the composer visibility button. See screenshot below.

In addition, this fixes the following issues:

  • Correctly handles editing and redrafting posts to include quoted posts.
  • On quote, the composer is focused like replies do.
  • Shows an alert when attempting to upload an image with a quote already set.
  • Makes visibility modal immediately update the UI but only persist on close.

Notes:

  • The legacy privacy dropdown is still used in the boost modal. This is because showing the new visibility modal clears the boost modal state, so it cannot be updated without refactoring to use Redux.
  • The visibility modal doesn't have the same styling for visibility as the legacy privacy dropdown. How this looks should be confirmed with design, but right now it's consistent with the quote policy dropdown design.
Screenshot 2025-08-20 at 17 32 17

@ChaosExAnima ChaosExAnima self-assigned this Aug 20, 2025
@ChaosExAnima ChaosExAnima marked this pull request as ready for review August 20, 2025 15:37
@ChaosExAnima ChaosExAnima force-pushed the feat/quote-post-composer-fixes branch 2 times, most recently from 5cce347 to 9ab5ae4 Compare August 20, 2025 15:46
@ChaosExAnima ChaosExAnima force-pushed the feat/quote-post-composer-fixes branch from 9ab5ae4 to f3c7ed8 Compare August 20, 2025 15:49
@ChaosExAnima ChaosExAnima requested a review from a team August 20, 2025 15:50
@ClearlyClaire
Copy link
Contributor

ClearlyClaire commented Aug 20, 2025

Apart of design points that are being discussed, there are a few edge cases that are not handled properly:

  • clicking the quote button when you already have a poll or media attachment gives focus to the composer but does not attach a quote, neither does it tell the user about it; this might be confusing
  • clicking the quote button when you already have a quote attached replaces the quote, even when you are editing a quote post, which will be confusing because we do not allow quotes to be changed

EDIT: solved

ClearlyClaire
ClearlyClaire previously approved these changes Aug 21, 2025
@ClearlyClaire ClearlyClaire added this pull request to the merge queue Aug 21, 2025
Merged via the queue into main with commit f85f0ee Aug 21, 2025
33 checks passed
@ClearlyClaire ClearlyClaire deleted the feat/quote-post-composer-fixes branch August 21, 2025 14:25
ClearlyClaire added a commit to glitch-soc/mastodon that referenced this pull request Aug 24, 2025
* Update dependency vite-plugin-pwa to v1.0.3 (mastodon#35830)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* New Crowdin Translations (automated) (mastodon#35829)

Co-authored-by: GitHub Actions <[email protected]>

* Status quote button (mastodon#35822)

* Use `debug?` query method on httplog initializer check (mastodon#35833)

* Fix display of quotes in threads in WebUI (mastodon#35834)

* Update dependency ruby-vips to v2.2.5 (mastodon#35836)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Disable `Metrics/*` cops (mastodon#35000)

* Update dependency core-js to v3.45.1 (mastodon#35843)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* New Crowdin Translations (automated) (mastodon#35844)

Co-authored-by: GitHub Actions <[email protected]>

* Update dependency vite-plugin-static-copy to v3.1.2 (mastodon#35846)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update dependency rubyzip to v3.0.2 (mastodon#35845)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Composer quote improvements (mastodon#35835)

Co-authored-by: Claire <[email protected]>
Co-authored-by: diondiondion <[email protected]>

* Refactor to reuse the one status partial across moderation tools (mastodon#35644)

* Fix layout shift caused by "Who to follow" widget (mastodon#35861)

* Improvements for keyboard navigation in feeds (mastodon#35853)

* Improve feed item focus outlines (mastodon#35864)

* Visibility Modal fixes (mastodon#35865)

* New Crowdin Translations (automated) (mastodon#35859)

Co-authored-by: GitHub Actions <[email protected]>

* [Glitch] Status quote button

Port 8268323 to glitch-soc

Signed-off-by: Claire <[email protected]>

* [Glitch] Fix display of quotes in threads in WebUI

Port 45ec4c9 to glitch-soc

Signed-off-by: Claire <[email protected]>

* [Glitch] Composer quote improvements

Port f85f0ee to glitch-soc

Co-authored-by: Claire <[email protected]>
Co-authored-by: diondiondion <[email protected]>
Signed-off-by: Claire <[email protected]>

* [Glitch] Refactor to reuse the one status partial across moderation tools

Port 4f49478 to glitch-soc

Signed-off-by: Claire <[email protected]>

* [Glitch] Fix layout shift caused by "Who to follow" widget

Port 511e10d to glitch-soc

Signed-off-by: Claire <[email protected]>

* [Glitch] Improvements for keyboard navigation in feeds

Port 118c30f to glitch-soc

Signed-off-by: Claire <[email protected]>

* [Glitch] Improve feed item focus outlines

Port a1c7b85 to glitch-soc

Signed-off-by: Claire <[email protected]>

* [Glitch] Visibility Modal fixes

Port 4df50b9 to glitch-soc

Signed-off-by: Claire <[email protected]>

---------

Signed-off-by: Claire <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: GitHub Actions <[email protected]>
Co-authored-by: Echo <[email protected]>
Co-authored-by: Matt Jankowski <[email protected]>
Co-authored-by: diondiondion <[email protected]>
Co-authored-by: Emelia Smith <[email protected]>
ronilaukkarinen added a commit to mementomori-social/mastodon that referenced this pull request Dec 26, 2025
mo-rijndael pushed a commit to mastodon-ml/mastodon that referenced this pull request Jan 25, 2026
mo-rijndael pushed a commit to mastodon-ml/mastodon that referenced this pull request Jan 25, 2026
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.

3 participants