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

Quote posts #52

Draft
wants to merge 59 commits into
base: dev
Choose a base branch
from
Draft

Quote posts #52

wants to merge 59 commits into from

Conversation

sneakers-the-rat
Copy link
Collaborator

@sneakers-the-rat sneakers-the-rat commented Jun 30, 2024

This is a ripoff of treehouse's quote posts.

image

upstream masto is going too slow in implementing quote posts and i want them. we can vote on merging this if we'd like to but i just want to get the patch working and prepared anyway.

So far what i've done is just cherrypick the commits from the two pull requests and any that have "quote" in the commit message....

So far that's a woefully inadequate job, so what i think this gets us is places to look for changes.

TODO

UI

  • don't crash when clicking quote while logged out https://types.pl/@jj/112862164894124697
  • visually offset quote more
  • make the whole quote area clickable - whole text area is clickable but it doesn't seem like it atm
  • the links don't work - for some reason there is a link to the profile page in the name and a link to the post in the body, but neither of them are clickable. this is probably bc the react component is intercepting the clicks and the click handler isn't working.
  • styling - add avatar and handle to the title bar
  • when drafting a quote post, the quote isn't displayed in the compose view, so it's not clear that you're quoting something. when you compose a reply, that is there with an 'x' box to allow you to cancel doing a reply, and I think that's supposed to be here but it's broken.
  • quotes are not displayed from the quoted post - ideally one would be able to see when they are quoted, but that doesn't seem to be shown now. should be a way to pop into a feed that shows the quotes of a post.
  • i think the truncation is good, but also glitch has that normally, so i'm not sure why we wouldn't just embed the quote as another Status component within the status component? probably some redux nightmare reason
  • quoted posts go below the quote post

Federation

  • Make them federate as a boost as well as a post, the OP should be notified and the quoted post should be federated to the other instances
    • rather than federating as a boost, just notify the person, it looks like a normal post to them d8fcac6

Importing from treehouse

make sure we got a complete import of prior work so we don't have a messy ass merge when we try and contrib back to them:

  • go through and take a look at all the other places that have changed in the treehouse code that weren't necessarily labeled as changes to quote posts and make sure this version matches theirs.
  • do a code search for 'quote' and make sure there aren't any new places that we don't have in this patch and import those.

i can already see a number of changes in other commits where quote stuff was changed along with other stuff, which is fine, they make no promises about feature portability or anything. anyway opening this up as a draft

@sneakers-the-rat
Copy link
Collaborator Author

sneakers-the-rat commented Jun 30, 2024

ok it working, but there are some bugs to fix and improvements to make. here is an example:
Screenshot 2024-06-30 at 1 31 56 AM

  • visually offset quote more: that whole text area is clickable but it doesn't seem like it atm, we should add a border and a bit more padding.
  • the links don't work - for some reason there is a link to the profile page in the name and a link to the post in the body, but neither of them are clickable. this is probably bc the react component is intercepting the clicks and the click handler isn't working.
  • styling - add avatar and handle to the title bar
  • when drafting a quote post, the quote isn't displayed in the compose view, so it's not clear that you're quoting something. when you compose a reply, that is there with an 'x' box to allow you to cancel doing a reply, and I think that's supposed to be here but it's broken.
  • quotes are not displayed from the quoted post - ideally one would be able to see when they are quoted, but that doesn't seem to be shown now. should be a way to pop into a feed that shows the quotes of a post.
  • i think the truncation is good, but also glitch has that normally, so i'm not sure why we wouldn't just embed the quote as another Status component within the status component? probably some redux nightmare reason

edit: moved these up to the OP

Copy link

github-actions bot commented Jul 5, 2024

This pull request has merge conflicts that must be resolved before it can be merged.

@sneakers-the-rat
Copy link
Collaborator Author

I think for the sake of this PR i'm gonna punt on a lot of the style problems and just try and get functional problems to work and then iterate in future PRs by putting the TODOs on the wiki before i merge

@sneakers-the-rat
Copy link
Collaborator Author

mentioning is just about that easy: d8fcac6

Screenshot 2024-07-27 at 11 23 39 PM 3c

@sneakers-the-rat sneakers-the-rat changed the base branch from main to dev August 11, 2024 06:22
kaniini and others added 21 commits August 18, 2024 21:13
(cherry picked from commit 56d4b04)
(cherry picked from commit 1df2577)
(cherry picked from commit a697e1d)
(cherry picked from commit 8d86c77)
kaniini and others added 29 commits August 18, 2024 21:13
(cherry picked from commit d23cd8d)
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.

4 participants