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

Soft deprecated #6915

Merged
merged 7 commits into from
Nov 14, 2020
Merged

Soft deprecated #6915

merged 7 commits into from
Nov 14, 2020

Conversation

AlicVB
Copy link
Contributor

@AlicVB AlicVB commented Nov 14, 2020

this close #6912 , this fix #6907
Here is some refinement on top of @TurboGit PR #6912

it create a special modulegroups preset with all recently deprecated modules.
So this modules are still accessible until next release when they will disappear from modules groups, except for old edit using them.
That way, it let some time to users to adapt their workflow accordingly.

Warning messages are show on top of each modules, with a proposal of replacement modules.

@aurelienpierre : can I ask you to review all the deprecated message inside modules ? Most are just copy-paste of some of your comments around... eventually we can even add the reason of deprecation but I'm not sure it's worth the pain...
@TurboGit : thanks for the base implementation

@AlicVB AlicVB added difficulty: average some changes across different parts of the code base feature: enhancement current features to improve scope: UI user interface and interactions labels Nov 14, 2020
@AlicVB AlicVB added this to the 3.4 milestone Nov 14, 2020
Copy link
Member

@TurboGit TurboGit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, probably the last feature merge for 3.4. Thanks. I'll fix some strings where we don't want to introduce capital letters for consistency and we are all set. Thanks AlicVB for handling this so fast.

@TurboGit TurboGit merged commit 34ae2bf into darktable-org:master Nov 14, 2020
@@ -112,6 +112,11 @@ int flags()
return IOP_FLAGS_DEPRECATED | IOP_FLAGS_ONE_INSTANCE | IOP_FLAGS_PREVIEW_NON_OPENCL;
}

const char *deprecated_msg()
{
return _("This module is deprecated.");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use color mapping module instead

(color mapping is basically color transfer v2)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see PR #6929 thanks for the info !

@chhil
Copy link

chhil commented Nov 15, 2020

Git pull : e820499..c8f2d6c master -> origin/master

Build 3.3.0+1613~gc8f2d6cd9

Is this warning correct with the modules listed under it?
This was an image that was imported prior to this build but was not edited.

image

@AlicVB
Copy link
Contributor Author

AlicVB commented Nov 15, 2020

it's a bug... that'll be fixed soon :)

@ralfbrown
Copy link
Collaborator

In addition to what chhil pointed out, every non-deprecated module now has a blank space just below its title bar when expanded.
(3.3.0+1618~g831906b21)

@AlicVB
Copy link
Contributor Author

AlicVB commented Nov 15, 2020

both bugs should be fixed now in master. Thanks for the report !

@ralfbrown
Copy link
Collaborator

fix confirmed for both bugs

@chhil
Copy link

chhil commented Nov 16, 2020

Just got this.... current master

3.3.0+1652~gf4d6d9920

image

@TurboGit
Copy link
Member

Expected as you are on the deprecated module group if I see correctly the top of the image.

@parafin
Copy link
Member

parafin commented Nov 16, 2020

But how it looks is very misleading...

@AlicVB
Copy link
Contributor Author

AlicVB commented Nov 16, 2020

@chhil : seems like a bug. Lens is not deprecated and you doesn't seems to be on the deprecated preset... You have only 1 group shown... I guess you have set the pref to only show search box. right ? if so, I'm currently fixing this !

@chhil
Copy link

chhil commented Nov 16, 2020

I had a clean fresh install, and that big on/off button looked odd.

I had got a similar thing for crop (typed crop in the search box) too when I pasted the image with a red label, you could check that too.

I have show both (module groups and search).

@AlicVB
Copy link
Contributor Author

AlicVB commented Nov 16, 2020

but in that case, why do you have only the "active modules" group ?
you've done your own preset ?

@chhil
Copy link

chhil commented Nov 16, 2020

I think when I install there is no selection and it only shows active modules.

image

Once I select a group one then there is no way to get that big central active modules.

After selecting "all modules"
image

@AlicVB
Copy link
Contributor Author

AlicVB commented Nov 16, 2020

So... you've found 2 bugs in one it seems :) I'll take care !

@chhil
Copy link

chhil commented Nov 16, 2020

Do check @aurelienpierre s new modulegroup pteset fix , may be related.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: average some changes across different parts of the code base feature: enhancement current features to improve scope: UI user interface and interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Softer module deprecation
5 participants