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

Make Your Selection: Mask #284

Open
ShelleyJ opened this issue Mar 15, 2018 · 1 comment
Open

Make Your Selection: Mask #284

ShelleyJ opened this issue Mar 15, 2018 · 1 comment

Comments

@ShelleyJ
Copy link

Your DownThemAll! and Browser Version and Language and Operating System

DTA version: 3.0.8
Firefox esr 52.7.0
Language: English
OpSys: Windows 7

Expected Behavior

When I check "Use mask only this time," I do not expect the dropdown list to add my ONE-TIME mask to the list and eliminate one of the masks that I use all the time. I expect the ONE-TIME mask to be used ONLY ONE TIME!!!!

Actual Behavior

The ONE-TIME mask is added to the dropdown list, and one of my frequently used masks is deleted!!!

Steps To Reproduce

Add a mask in the the "Mask:" box, check "Use mask only this time"

Additional info such as log extracts

Please enable Diagnostic Logging in the Privacy tab of the DownThemAll! preferences and either copy/paste essential part of the log or attach the entire log. Please note that the log may include sentitive information which you have to remove prior to posting it!

@ShelleyJ
Copy link
Author

ShelleyJ commented Mar 15, 2018

Since I download frequently, and your add-on doesn't allow me to uniquely identify files (NOT a choice in your "Renaming Tags"), I must create my own mask for my downloads. Since this mask is UNIQUE to each particular download situation, I want to USE THIS MASK ONLY ONE TIME!!! And I do not want to delete the masks I use frequently from the list!!!

Since I have contributed to your development coffers, I would consider it a favor if you fixed this problem.

Since I am a software development professional myself, I'm fairly certain that this fix would not require more than a couple of lines of code... for example: If the "Use mask only this time" is checked, Jump over the "Update dropdown list subroutine" OR Don't CALL the "Update dropdown list subroutine."
Two lines of code to fix this problem.

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

No branches or pull requests

1 participant