WIP:Supersedes deprecated function calls #43

Open
obache wants to merge 1 commits from feat/supersedes-deprecated into master
obache commented 4 months ago
Collaborator

No reason to use TDE core functions marked as DEPRECATED in TDE core modules.

No reason to use TDE core functions marked as DEPRECATED in TDE core modules.
obache added 1 commit 4 months ago
5e8185513d Supersedes deprecated function calls
Poster
Collaborator

setButtonOKText and setButtonCancelText` usage might be wrong originally.
It seems mismatching "text", "tooltip" and "whatsthis" combination.

`setButtonOKText` and setButtonCancelText` usage might be wrong originally. It seems mismatching "text", "tooltip" and "whatsthis" combination.
Owner

Hi Obata-san, I take this is still WIP since you have mark it as such in the title. Correct?

Hi Obata-san, I take this is still WIP since you have mark it as such in the title. Correct?
Poster
Collaborator

kolourpaint/kptool.cpp:
TODO about ALT should be resolved, but not sure.

kpovmodeler/pmerrordialog.cpp:
OK butttun text still "OK", but "WhatsThis" is "When clicking Proceed", mismatch?
Cancel button, why "ToolTip" is prefixed with "&"?
Such question point is originally exists, but?

"setButtonOK()" will not update "WhatsThis", may different result.
https://mirror.git.trinitydesktop.org/gitea/TDE/tdelibs/src/branch/master/tdeui/kpushbutton.cpp#L134

kpovmodeler/pmpovraywidget.cpp:
"(TQWidget*)nullptr" is good here?

kolourpaint/kptool.cpp: TODO about ALT should be resolved, but not sure. kpovmodeler/pmerrordialog.cpp: OK butttun text still "OK", but "WhatsThis" is "When clicking Proceed", mismatch? Cancel button, why "ToolTip" is prefixed with "&"? Such question point is originally exists, but? "setButtonOK()" will not update "WhatsThis", may different result. https://mirror.git.trinitydesktop.org/gitea/TDE/tdelibs/src/branch/master/tdeui/kpushbutton.cpp#L134 kpovmodeler/pmpovraywidget.cpp: "(TQWidget*)nullptr" is good here?
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.