-
Notifications
You must be signed in to change notification settings - Fork 53
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
Grouping of profiles #167
Comments
I have sorting working in a local development version but need to clean it up before making a pull request for it. |
Right on, by chance are you able to also sort them into sub groups? I also
had sorting in my local branch but didn't attack sub groups yet for
organisational purposes related to issue #167. It isn't needed yet but I'll
eventually like to implement groups after sortibility.
…On Fri, Apr 20, 2018, 5:22 PM Jeremy Harnois, ***@***.***> wrote:
I have sorting working in a local development version but need to clean it
up before making a pull request for it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#167 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAjeHG1uunW0HT6idtJp5Lc9Gs0OAeYks5tqm23gaJpZM4S7Um3>
.
|
I didn't address grouping because I didn't feel it was necessary after I added sorting for myself. I'm also coming from Firefox with a lot of profiles, and sorting solved the major usability issues I was having. I'm curious to see if it does the same for @gwillegers. @gwillegers, how were you using groups? I think other features would be more useful and would, at least for me, solve the same problems I was using grouping to solve. IOW, grouping may have been used to solve usability issues that should probably be addressed with more specific solutions. Two features I'd find useful that would replace how I was using groups would be:
|
I created groups for things like work, finance, webshops, holidays, hobbies. IMO it makes it easier to find an account. Sorting helps, but sorting and grouping would be ideal. (BTW: the original PasswordMaker only had groups, no sorting.) |
Where's your grouping code? @gwillegers I'd like to review it and maybe add it. |
A misunderstanding must have slipped into this discussion. Grouping accounts is just part of the functionality of the original Passwordmaker extension for Firefox. You can create accounts, but you can also create groups, and create accounts in a group (just like files and directories, although I don't know if nested groups are possible). I did not code it myself. I just used it. |
Oh OK @gwillegers I was under thinking you had it coded, I should give it a shot soon. |
The lack of sorting was my biggest complaint about the old Firefox extension; grouping only helped so much. @gwillegers, when are you manually looking for an account instead of relying on pattern matching to automatically select the account for you? I was considering a filter-as-you-type feature—both in the popup and options—as a way to more easily find accounts when I have to look for them manually. While that's a different approach than what you're used to, I feel like that would be a better (easier, quicker) feature than grouping. |
Looking for an account is mostly when I want to change something. Some passwords have to be changed periodically, or the webiste's URL changes (so I need to change the patterns). Most of the time the pattern matching is sufficient to select the right account. |
Hey @jeremy-harnois if you like to test out my filter feature, try pulling from my master branch and see if it works for you 😄 |
I pushed out version 0.8.8 with the profile filter and alphanumeric sorting features for you both to try out @jeremy-harnois and @gwillegers |
Works great! Filtering is a nice alternative for grouping. |
I submitted new version to Mozilla. Waiting for approval. |
Well I'm glad filtering seems to be working OK for people. |
Thanks @heavensrevenge ... now I can use a legitimate release :-) |
Right on @jeremy-harnois, I wonder if its a suitable alternative for grouping though, if so, either @gwillegers or me can close this issue. |
So @jeremy-harnois how are you liking it so far? |
I like it; seems to work well, @heavensrevenge. Filtering also solves an issue I was having with not being able to see the selected profile's details when selecting a profile after about the 50th one unless I scrolled back to the top. I don't feel there's a need for grouping, but that's @gwillegers request, so I'll let him comment on whether or not he thinks it's still needed after he's had some more time to work with filtering and sorting. |
That's good to hear, I was curious on feedback after people have had some time to play with it. Filtering was SO much more simple and clean to implement vs grouping and even drag & drop sorting. So I'd be happy if I didn't need to add drag & drop or groups to not add ~ 200 lines of code lol |
@jeremy-harnois I believe I fixed and optimized regex profile matching which I based on your ~5yo branch but didn't need to add another option field or profile entry via commit c0a55af. |
c0a55af also fixes #186 (and its duplicate #185), @INeedCaffeine 👍 |
As a user of the original PasswordMaker Firefox extension, I have a list of over 200 profiles. They are organised in groups. It would be nice to be able to create (sub-)groups to organise the profiles. Sorting the groups and profiles would also be nice to have.
The text was updated successfully, but these errors were encountered: