Developer forums (C::B DEVELOPMENT STRICTLY!) > Plugins development

Keybinder v2 notes/problems

<< < (14/15) > >>

oBFusCATed:
I don't like it because there is not need to put workarounds in the code. It is fine if trunk/master is broken for some time. If someone is affected he/she can revert the commit that is causing the problem.

BTW: Connecting events to the parent is not a good idea in my opinion. It is too fragile if someday we decide to change the UI of the config dialogs.
I'm talking about the page-changed and destroyed events. For the latter - why do you needed? Can't you use the cancel/apply methods implementations?

If there is something missing from the API it is better to add it. Applying workarounds leads to more work in the future.

Pecan:

--- Quote from: oBFusCATed on April 09, 2020, 07:16:26 pm ---I don't like it because there is not need to put workarounds in the code. It is fine if trunk/master is broken for some time. If someone is affected he/she can revert the commit that is causing the problem.

BTW: Connecting events to the parent is not a good idea in my opinion. It is too fragile if someday we decide to change the UI of the config dialogs.
I'm talking about the page-changed and destroyed events. For the latter - why do you needed? Can't you use the cancel/apply methods implementations?

If there is something missing from the API it is better to add it. Applying workarounds leads to more work in the future.

--- End quote ---

I will no longer need the events now that OnPageChanging() is provided.

I used the events to avoid the large overhead caused by scanning the menu structure and building the voluminously stacked arrays used by the original keybinder code from 2005.

The overhead occurred when the user chose Settinging/Editor. There was no need for that overhead until the user actually chose "Keyboard shortcuts."  Using the events cut that overhead in half.

The event method served the same purpose that is now provided by OnPageChanging(). I had already planned to remove the events and use that UI instead.

The duplicate configurationpanel.h was a fix for disappearing wxMessageBoxes on linux from years and years ago. It also will be removed as it's no longer needed.
 

gd_on:
A new problem, mix of languages :
Usually, I use a translated version of C::B, in french. Sometimes, I need to switch the interface to English, at least to verify the exact word used in the original interface.
After my last switch, I have seen this dialog (see attachment). The sentence, describing the conflict in English and french are the same, as if there where a mix of languages.
(svn 12069, but may be appeared before).
And in Settings/Environmement/Keyboard Shortcuts Ctrl-W and Ctrl-Shift-W appears twice.
I tried to delete one of them in the right part, but it's not sufficient. I have to delete All to avoid this warning.
gd_on

Pecan:

--- Quote from: gd_on on April 21, 2020, 10:58:40 am ---A new problem, mix of languages :
Usually, I use a translated version of C::B, in french. Sometimes, I need to switch the interface to English, at least to verify the exact word used in the original interface.
After my last switch, I have seen this dialog (see attachment). The sentence, describing the conflict in English and french are the same, as if there where a mix of languages.
(svn 12069, but may be appeared before).
And in Settings/Environmement/Keyboard Shortcuts Ctrl-W and Ctrl-Shift-W appears twice.
I tried to delete one of them in the right part, but it's not sufficient. I have to delete All to avoid this warning.
gd_on

--- End quote ---

Until I can re-create and find a fix for this, using separate personalities for differenct languages will solve the problem.

See the CB /p parameter.

Pecan:

--- Quote from: gd_on on April 21, 2020, 10:58:40 am ---A new problem, mix of languages :
Usually, I use a translated version of C::B, in french. Sometimes, I need to switch the interface to English, at least to verify the exact word used in the original interface.
After my last switch, I have seen this dialog (see attachment). The sentence, describing the conflict in English and french are the same, as if there where a mix of languages.
(svn 12069, but may be appeared before).
And in Settings/Environmement/Keyboard Shortcuts Ctrl-W and Ctrl-Shift-W appears twice.
I tried to delete one of them in the right part, but it's not sufficient. I have to delete All to avoid this warning.
gd_on

--- End quote ---

I am unable to find a translation to download and test with.
Launchpad gives me an "Invalid OpenID transaction" error and https://translations.launchpad.net/codeblocks-gd wants a UbuntuOne login.

Does anyone have a translation I can download.
Also, I once found instruction for using translations but now can't find it. Does anyone know where there are instructions for using translations.

TIA

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version