Repository navigation
Part 2: 638 Feature Request: password generator with options - #873
Conversation
The actual options don't quite work yet.
Why is this menu still wrong...
638 - Another...
638 - PR Concerns
| AntiAlias: 1 | ||
| FitToEm: 0 | ||
| WinInfo: 0 38 14 | ||
| WinInfo: 0 33 12 |
There was a problem hiding this comment.
Not sure about some of these extra changes and whether they're necessary.
|
Hi @veler, it looks like you're full steam ahead with 2.0. I am mostly done adding things I was missing when I first started using DevToys. This PR is just to clean up an old one that was hanging around. Do you have any plans to release a new version of 1.0 soon? Just to hold us over until 2.0 is maybe ready for early testing? 😊 |
Hi @micahmo , I am so sorry for the delay. I actually started looking at the PR weeks ago and then completely forgot. I will definitely release the contribution you made soon, before DevToys 2.0. :) Thank you for this pull request. It looks all good to me. I don't have much comment to make as they were already addressed from the previous PR. |
Pull request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: #638
What is the new behavior?
This is a continuation of the work started in #664.
Other information
I believe I have addressed the only two outstanding comments from the original PR which were...
Adding a new icon: I have added the icon suggested by @veler from the FluentSystemIcons font.
Supporting numbers as a toggle: I have added the numbers setting to the UI, and an
InfoBar(Warning) is shown if no characters sets are selected.Quality check
Before creating this PR: