Skip to content

Super dev mode for version 2.10.0 is disabled and radio button is hidden #459

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

Conversation

safy97
Copy link
Contributor

@safy97 safy97 commented Mar 1, 2023

When importing GWT SDK 2.10.0, I found that radio button group that allows to choose between super development mode and legacy developement mode in the GWT debugging configuration is hidden and argument -nosuperdevmode is forced.

I found that the pattern that decide whether the GWT version supports super dev mode or not excludes 2.10.0.

I know that 2.10.0 still not supported by current plugin but as you see as the code support versions [3-9].* it should also support 2.10.0 version.

@protoism
Copy link
Contributor

protoism commented Mar 1, 2023

Hi @safy97, the PR looks quite focused and correct. I checked here and there are no regression.
Just to be super safe, did you test it with Eclipse?

@safy97
Copy link
Contributor Author

safy97 commented Mar 1, 2023

Hi @protoism Thank you for your fast reply, yes i tested it with eclipse and let me know if i should do anything from my side.

@protoism protoism added this to the 4.0.1 milestone Mar 2, 2023
@protoism
Copy link
Contributor

protoism commented Mar 2, 2023

The PR is quite simple and reasonable.
Approved

@@ -744,7 +744,7 @@ protected void updateLaunchConfigurationDialog() {
* @return true if super dev mode can be used
*/
private boolean canSdkVersionUseSuperDevMode(String version) {
return version.matches("^2.[5-9].*") || version.matches("^[3-9].*");
return version.matches("^2.([5-9]|1\\d).*") || version.matches("^[3-9].*");
Copy link
Contributor

Choose a reason for hiding this comment

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

OK

@protoism protoism merged commit ee474e5 into gwt-plugins:main Mar 2, 2023
@protoism
Copy link
Contributor

protoism commented Mar 2, 2023

Hi @protoism Thank you for your fast reply, yes i tested it with eclipse and let me know if i should do anything from my side.

Contribute more :)

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

Successfully merging this pull request may close these issues.

2 participants