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

implemented generation of anon names #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

VcSaJen
Copy link

@VcSaJen VcSaJen commented Jul 1, 2020

Before you begin

  • I understand my contributions may be rejected for any reason
  • I understand my contributions are for the benefit of Ponybooru and/or the Philomena software
  • I understand my contributions are licensed under the GNU AGPLv3
  • I understand all of the above

So, there it is. Generator for anon/bp names like "Western Drums #F447" or "Worrisome Door #FB44". It's in the server code, not in JS, so it would work for users with disabled JS. Do you think it's worth it to add? Or not? I'm fine with either way.
One thing to note: if users reply "@Brilliant Marmalade #1582" in their comments, it will remain so regardless of anything, because it's a static text.
Performance is not much different than SVG Background Pony avatar generation, because it was based on that code.

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.

1 participant