Skip to content

feat(plugin): GifCollections#461

Closed
Syncxv wants to merge 62 commits into
Vendicated:mainfrom
Syncxv:feat/gifcollections
Closed

feat(plugin): GifCollections#461
Syncxv wants to merge 62 commits into
Vendicated:mainfrom
Syncxv:feat/gifcollections

Conversation

@Syncxv
Copy link
Copy Markdown
Contributor

@Syncxv Syncxv commented Jan 29, 2023

basically it lets you organize your gifs / images into collections

would love some feedback and improvements

Discord_rforZhGndC.mp4

todo:

  • option to remove a collection
  • option to remove a gif from collection
  • use data store instead of localstorage
  • figure out width and height of gifs :|
  • support vidoes
  • fix the flash that happens in the gif viewer thingy
  • add context menu for messages with multiple vidoes
  • option to export your collections
  • option to import your collections
  • add collection context menu in the gif picker

Syncxv and others added 12 commits January 29, 2023 23:19
+using data store
+Collection Manager
+latest gif / collection will come first
+uuidv4 for gifs

and some smol changes
and fix context menu apearering on trending sheet
gonna figure out forceupdate now
now gotta make it work for modal thingy yk when ya click on the image
lets just call it a day
@Vendicated
Copy link
Copy Markdown
Owner

hi, is this done yet?

@Syncxv
Copy link
Copy Markdown
Contributor Author

Syncxv commented Feb 1, 2023

yeah its finished :D

i forgort to say it finished was whoops

@Vendicated
Copy link
Copy Markdown
Owner

okay cool, wasn't so sure because your last commits all sound like this is a WIP

@Vendicated
Copy link
Copy Markdown
Owner

this is a really cool plugin btw, but now write an AI that will sort my ~200 favourited gifs into collections

@Syncxv
Copy link
Copy Markdown
Contributor Author

Syncxv commented Feb 1, 2023

ayo

Comment thread src/plugins/gifCollections/CollectionManager.ts Outdated
Comment thread src/plugins/gifCollections/CollectionManager.ts Outdated
Comment thread src/plugins/gifCollections/CollectionManager.ts Outdated
Comment thread src/plugins/gifCollections/index.tsx Outdated
Comment thread src/plugins/gifCollections/utils/uuidv4.ts Outdated
Comment thread src/plugins/gifCollections/utils/getUrlExtension.ts Outdated
Comment thread src/plugins/gifCollections/utils/getFormat.ts Outdated
Comment thread src/plugins/gifCollections/index.tsx Outdated
Comment thread src/plugins/gifCollections/index.tsx Outdated
Co-authored-by: Ven <vendicated@riseup.net>
@Syncxv
Copy link
Copy Markdown
Contributor Author

Syncxv commented Feb 1, 2023

ok there seems to be an issue with the menu. target is undefined so i cant get the height and widths of the gif. ima try fix it gimme a sec

Syncxv and others added 8 commits February 1, 2023 23:58
Co-authored-by: Ven <vendicated@riseup.net>
Co-authored-by: Ven <vendicated@riseup.net>
Co-authored-by: Ven <vendicated@riseup.net>
Co-authored-by: Ven <vendicated@riseup.net>
theres 100% a better way to do this but its good for now ig
Syncxv and others added 26 commits February 12, 2023 07:15
it only use to work for vidoes you uploaded
now it works for vidoes you just paste as a link yk

also add predicate for menu item near open link
+add collection context menu on gifs in the gif picker
+fix multiple add collection menu items on tennor gifs

rename createGif.ts -> getGif.ts

so how was your day
and fix oopsie with get format
- improved patches
- using context menu api
- improved getGifByMessageAndUrl
- moved makeMenu to getGif()
- add return types collection manager to make it more consistent
@Syncxv Syncxv closed this Jul 16, 2023
@VillainsRule
Copy link
Copy Markdown

@Syncxv why was this closed?

enzomtpYT pushed a commit to enzomtpYT/PawsomeVencord that referenced this pull request Oct 28, 2025
…d#461)

* feat: Add AutoZipper to zip folders/configured file types. (and add myself to devs)

* Remove logging for event listener attachment

---------

Co-authored-by: thororen <78185467+thororen1234@users.noreply.github.com>
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