Conversation

Notices

1. Embed this notice
   meso (meso@asbestos.cafe)'s status on Friday, 10-Feb-2023 05:34:35 JST meso

   • Alex Gleason
   • Mercurial
   @alex why :( this is not good for my boost power usering cc @MercurialBlack
   • Embed this notice
     Alex Gleason (alex@gleasonator.com)'s status on Friday, 10-Feb-2023 05:34:34 JST Alex Gleason

     in reply to

     • Mercurial

     @meso @MercurialBlack Hold shift while clicking it and it’ll repost immediately without the menu

   • Embed this notice
     Matty (matty@nicecrew.digital)'s status on Friday, 10-Feb-2023 05:40:37 JST Matty

     in reply to

     • Alex Gleason
     • Mercurial
     >mobile focused UI design
     >hotkeys
     
     Alex
   • Embed this notice
     Alex Gleason (alex@gleasonator.com)'s status on Friday, 10-Feb-2023 05:40:37 JST Alex Gleason

     in reply to

     • Matty
     • Mercurial

     @matty @meso @MercurialBlack This is what users want the UI to look like.

   • Embed this notice
     Alex Gleason (alex@gleasonator.com)'s status on Friday, 10-Feb-2023 05:43:19 JST Alex Gleason

     in reply to

     • ew
     • Mercurial

     @e @meso @MercurialBlack Yes. There’s no good hotkey library for React.

   • Embed this notice
     ew (e@masochi.st)'s status on Friday, 10-Feb-2023 05:43:20 JST ew

     in reply to

     • Alex Gleason
     • Mercurial
     @alex @meso @MercurialBlack >hold shift
     is the react hotkey library situation still garbage?
   • Embed this notice
      (mint@ryona.agency)'s status on Friday, 10-Feb-2023 05:43:41 JST 

     in reply to

     • 
     • Alex Gleason
     • Mercurial

     @MercurialBlack @alex @meso This works.

     diff --git a/app/soapbox/utils/features.ts b/app/soapbox/utils/features.ts index 59ef9f4ad..7c3a34179 100644 --- a/app/soapbox/utils/features.ts +++ b/app/soapbox/utils/features.ts @@ -672,6 +672,7 @@ const getInstanceFeatures = (instance: Instance) => { */ quotePosts: any([ v.software === PLEROMA && [REBASED, AKKOMA].includes(v.build!) && gte(v.version, '2.4.50'), + features.includes('quote_posting'), instance.feature_quote === true, ]),

     Might be worth removing Rebased from version check as well, since I think all instances of it have quote_posting defined in features, only Akkoma acts like a special needs child in this regard. In either case, this would be of use for insane people like me who merged quote-post branch of upstream Pleroma instead of migrating. Too lazy to registed on Gitlab and make a proper pull request, sorry.

     Alex Gleason likes this.
   • Embed this notice
      (mint@ryona.agency)'s status on Friday, 10-Feb-2023 05:43:42 JST 

     in reply to

     • 
     • Alex Gleason
     • Mercurial
     @meso @alex @MercurialBlack Yeah, it doesn't check the feature list, instead only checking the version string plus separate `feature_quote` field from Fedibird.
     https://gitlab.com/soapbox-pub/soapbox/-/blob/develop/app/soapbox/utils/features.ts#L667
   • Embed this notice
      (mint@ryona.agency)'s status on Friday, 10-Feb-2023 05:43:44 JST 

     in reply to

     • Alex Gleason
     • Mercurial
     And I somehow don't get to choose, it goes straight to repost confirmation despite having `quote_posting` in metadata features list.
   • Embed this notice
     Alex Gleason (alex@gleasonator.com)'s status on Friday, 10-Feb-2023 05:44:17 JST Alex Gleason

     in reply to

     • 
     • Mercurial

     @mint @meso @MercurialBlack Ultimately we want this, but it got put a bit on the backburner. https://gitlab.com/soapbox-pub/soapbox/-/merge_requests/1914