Skip to content

Remove deprecated properties. - #15898

Closed
ghost wants to merge 3 commits into
masterfrom
unknown repository
Closed

Remove deprecated properties.#15898
ghost wants to merge 3 commits into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Dec 18, 2023

Copy link
Copy Markdown

Hello everyone,

both properties are deprecated. I just removed them, or replaced them with other methods. The only thing I see as a result, is the bigger checkbox here on a high definition display. I would say, they now have the right size, like the other checkboxes. I cannot test how this looks on different displays or scales.

Maybe someone else can test this code.

Thanks in advance.
Greetings

@TurboGit TurboGit added the gtk4 label Dec 19, 2023
@TurboGit TurboGit added this to the 4.8 milestone Dec 19, 2023
@TurboGit

Copy link
Copy Markdown
Member

When you see a diff on the UI it would be nice to have some screenshots. Easier for everyone to understand the actual change. TIA.

@ghost

ghost commented Dec 21, 2023

Copy link
Copy Markdown
Author

When you see a diff on the UI it would be nice to have some screenshots. Easier for everyone to understand the actual change. TIA.

I will keep that in mind for the next time.

Thanks.

@ghost ghost mentioned this pull request Dec 21, 2023
@zisoft

zisoft commented Mar 5, 2024

Copy link
Copy Markdown
Collaborator

The only thing I see as a result, is the bigger checkbox here on a high definition display.

I tested this PR on macOS and Windows and I cannot see any difference in the checkboxes.

@TurboGit TurboGit modified the milestones: 4.8, 5.0 May 13, 2024
@ghost

ghost commented Nov 23, 2024

Copy link
Copy Markdown
Author

In order to make the whole migration inside a single branch, close this one and merge the branch Christian-Kr:krippendorf/gtk4_prepare_001 into Christian-Kr:krippendorf/gtk4.

This PR is not needed anymore.

Greetings

@ghost ghost closed this Nov 23, 2024
@TurboGit

Copy link
Copy Markdown
Member

@Christian-Kr : Thanks!

This pull request was closed.
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.

2 participants