gyng / save-in

WebExtension for saving media, links, or selections into user-defined directories
MIT License
205 stars 25 forks source link

Context Menu display name alias #64

Closed DanaMW closed 6 years ago

DanaMW commented 6 years ago

I was wondering how you might feel about a request for an alias for folder names on the context menu? If the path is long to a given folder it might be useful to be able to have it display as a user configured alias. using setup similar to others you use it could be added to the menu maybe . Pictures:Pictures: Pictures/2018/Family/Child:ChildName:

Or on the next line maybe? Just a thought to keep my menu reasonable.. Thank you friend.

gyng commented 6 years ago

Need to think of a nice way to do it without overloading the syntax.

gyng commented 6 years ago

I've added aliases to v3.1.0, to use them add (alias: display name) to the line's comments.

cats // (alias: actually dogs)
DanaMW commented 6 years ago

Thank you for your work great update there @gyng

DanaMW commented 6 years ago

I created a New menu, it was not a actual folder. and it caused the menus to spool one after another. I assumed the menu head could be an alias and not an actual. Alias renaming works perfectly 2018-08-04

DanaMW commented 6 years ago

Only two menus down sit right in Firefox the rest continue to do that, at least for me. My set up is like this is this correct? (names changed to protect the porn, I mean names of the innocent) I tried it both ways below and had the same result And I put the head menu on a real one of course. so Unless I am doing something wrong it is not working in FF Developers.

.                                                                          .
---
NewOne
One
>One01/Two01
>One02/Two02
>One03/Two03
>One04/Two04
>One05/Two05
>One06/Two06
>One07/Two07
>One08/Two08
>One09/Two09
>One10/Two010
>One11/Two011
>One12/Two012
>One13/Two013
>One14/Two014
>One15/Two015
>One16/Two016
>One17/Two017
>One18/Two018
>One19/Two018
>One20/Two020
>One21/Two021

.
---
NewOne
>One
>>One01/Two01
>>One02/Two02
>>One03/Two03
>>One04/Two04
>>One05/Two05
>>One06/Two06
>>One07/Two07
>>One08/Two08
>>One09/Two09
>>One10/Two010
>>One11/Two011
>>One12/Two012
>>One13/Two013
>>One14/Two014
>One15/Two015
>One16/Two016
>One17/Two017
>One18/Two018
>One19/Two018
>One20/Two020
>One21/Two021
gyng commented 6 years ago

I thought I had fixed that in dev, updated with v3.1.1.

DanaMW commented 6 years ago

Thank you and installed and working

DanaMW commented 6 years ago

This is working fantastic and if i could I would like to close this issue. I am ALL good. Thank you.