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

Add Post Titles to Filenames #608

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MetalNeo
Copy link
Contributor

Currently, filenames that are currently used by default do not stand out very much, especially when part of a large gallery. So, the rules have been updated to include the post title at the beginning of the filename to make individual files stand out more. In addition, gallery indexes have been added to filenames to ensure gallery images maintain their original order.

@MetalNeo
Copy link
Contributor Author

I just noticed is that these new rules do not work if the post is viewed from the comments view. Same applies to my recent #584 PR. Will need to perform some additional troubleshooting/testing on this one. Maybe also get comments working so #581 can be closed out.

@MetalNeo
Copy link
Contributor Author

I think I'll put the comment filename idea on the backburner for now. I'm just going to leave the PR as is and have #581 closed if it gets merged. The NSFW rules still need updated to work on the comments view though.

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.

None yet

1 participant