Add quoting by url / in replies #3488
No reviewers
Labels
No labels
BE Request
Backlog
Better behavior
Blocked by deps
Bug
Code Review
Discussion
Doing
Feature Request / Enhancement
In the shining bright future maybe
It's complicated
MS Edge
Mememoon (or other niche browsers)
Missing API
Missing feature
Missing l10n/i18n
NL1
NL2
NL4
Need to verify on develop
RELEASE BLOCKER
Reassign or Close
Refactor
Regression
Safari
To Do
User story
Waiting on godot
accessibility
better documentation
chore
confirmed
easy ticket
incident
l10n update
mastoapi
mobile
needs design
needs-info
needs-review
stupid
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
pleroma/pleroma-fe!3488
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "iamtakingiteasy/pleroma-fe:gitlab-mr-iid-2164"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Re-created (now deleted/lost) https://git.pleroma.social/pleroma/pleroma-fe/-/merge_requests/2204 (
which will probably start referencing some unrelated changes in the future, when MR ID is re-usednot since gitlab is gone and numeration jumped to 3k)Re-created (botched by migration from gitlab) #3431
Current quoting UI implementation only allows to embed a quote in the first post of a thread, referencing another post in replies in federation-compatible way is not possible. Aside from main limitation, current UI is also sticking out disproportionately, at least in the default theme. This MR aims to address both issues by moving quoting controls to it's own form activated by a status action button, similarly to polls, allowing to quote-by-url and choose whether to start a new thread with quote-reply. URL input accepts both /notice/ and resolvable APID links via search v2 API.
description (v1)
Additionally a convenience setting "When quoting, quote replied post in a new thread by default" is added;
Changes
Replaces quote/reply radio buttons with more subduedquote status action button revealing form with quote url input,start-new-thread checkboxand quote preview.Adds a new setting to control whether quote form should be prefilled with replied post url and start-new-thread prechecked.Errorsusing constructor cause field, rather than otherwise ignored object.Changes
Errorsusing constructor cause field, rather than otherwise ignored object.As side-effect of using search v2 API, technically a unique combination of terms may also be used to lookup the post to quote.
screenshots (before)
Before:screenshots (v1)
After:screenshots (v2)
After:702dcc8292to7b26819abc"start new thread" combined with a link is a bit confusing. I'd suggest keeping existing buttons + add "quote" icon next to poll one that expands into field "quote post by url" or something
@hj Won't this be even more confusing, if user selects "Quote this status" in a reply, then provides a link to another post, or provides link to a post, then selects "Quote this post"?
We could try to work-around that with disables: disable quote icon if "Quote this status" is selected and disable "Quote this status" if link/text is provided in a quote field, would you prefer that to the proposed "Start new thread" toggle? If nothing else, with "Start new thread" no non-obvious disables are necessary, everything is as presented.
Also just to clarify: quote icon next to poll expanding the quote form is already present, see screenshots.
Yeah disabling one or the other is right way.
7b26819abcto7aefda4211Done, updated screenshots and MR description.