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

Small improvements to the Prepare > Check Data > Duplicates dialog #7354

Conversation

derekagorhom
Copy link
Contributor

@derekagorhom derekagorhom commented Apr 4, 2022

Fixes #7299
Small changes in the duplicates Dialogue

Selected Variables has become the first button, and hence the default when you first use the dialogue. The Data Frame is then second. The Successive values remains the last button.
And the pull down has the first entry as Index.

@Patowhiz and I will make a new PR for the database changes recommended by @lloyddewit in #7305

Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

I really did think this was only a small task:
image
I am keen that the option titled Index Number of Duplicates should have Index as the default. It still has Count.
I changed it to Index and it works fine. Then the screen shot show what happens when I return. It has changed back to Count. It is supposed to remember the previous setting.
I notice you have changed Index to be the first of the 2 options, where it was Count before. That's fine, but it isn't sufficient.

@derekagorhom
Copy link
Contributor Author

Hello @rdstern i have done all these changes on the PR, but they do not show because of the database Translation, that is why we will make a PR for the database translation to correct this PR

@Patowhiz
Copy link
Contributor

Patowhiz commented Apr 7, 2022

@lloyddewit @rdstern, @derekagorhom and I have created PR #7363 that contains the required translation changes. Once merged this will be ready for review.
Thanks

@Patowhiz
Copy link
Contributor

Patowhiz commented Apr 7, 2022

@derekagorhom you may want to create a new issue about how the designer class name of the dlgDuplicates is named asdlgDuplicateRows. It's recommended to have the 'designer' class name and 'code' class be the same. You can also do the name changes in this PR or in another PR. Of course the rename will want a slight translation change too.
Thanks.

Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

@derekagorhom and @Patowhiz I don't understand why this change is causing so many problems.
image

In this option Count is still given as the default (Though Index is the first of the two options.
And if I change to Index, then it does it fine. But when I return to the dialogue it still has returned to count.

If this continues, perhaps instead we should have a 4th option for the radio button. But I really don't know why Index couldn't be the default, and why it can't remember the setting. That's what we usually do?

@derekagorhom
Copy link
Contributor Author

@rdstern sorry for the delay, PR #7363 will solve the translation issues with this PR

@Patowhiz
Copy link
Contributor

@rdstern you may have to wait for the translation PR #7363 to be merged. @lloyddewit is aware of that too.

@derekagorhom
Copy link
Contributor Author

@rdstern could you test this again, i merged the approved PR #7363 to this and it seems to work fine now

rdstern
rdstern previously approved these changes Apr 15, 2022
Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

Great!

instat/dlgDuplicates.vb Outdated Show resolved Hide resolved
@lloyddewit
Copy link
Contributor

@rdstern I made a small change that should not affect the functionality. If you test/approve again, then we can merge.
Thanks!

Copy link
Collaborator

@rdstern rdstern left a comment

Choose a reason for hiding this comment

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

Great

@lloyddewit lloyddewit changed the title Fixes on Duplicates Dialogue Small improvements to the Prepare > Check Data > Duplicates dialog May 3, 2022
@lloyddewit lloyddewit merged commit e53b640 into IDEMSInternational:master May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Small changes in the Prepare > Check Data > Duplicates dialogue
5 participants