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

Editor Tweaks plugin

<< < (3/7) > >>

carra:
Hello, I would like to report a bug in the Aligner feature (very useful, by the way!) of Editor Tweaks.
I am now using a fresh copy of prebuilt Nightly build 7925 for Windows version. I have tested this on 2 machines, Vista and XP and the same happens.

Right after install, the Aligner option of the context menu provides 4 default alignment options, and lets the user create more of its own. This works fine: after creating them, the aligner menu shows all existing options and they work.

The bug comes when restarting C::B. It seems that the menu always loads only 4 entries. I have verified that even when default.conf contains more entries, the menu always shows only 4, and they may not be the default ones. Not a critical bug, but rather annoying!

MortenMacFly:

--- Quote from: carra on April 24, 2012, 08:40:35 pm ---the menu always shows only 4, and they may not be the default ones. Not a critical bug, but rather annoying!

--- End quote ---
It is hard coded in EditorTweaks.cpp:

--- Code: ---    const unsigned int MaxStoreAlignerdEntries = 4;

--- End code ---
The reason might be that dmoore didn't want to clutter the editor menu too much.However, if you compile C::B yourself you can just change the value (or use one of the first 4 entries ;))

carra:
Well I don't compile C::B so I will have to use it as it is. It does seem weird though, that you are given the option to create more, and the code doesn't take this into account.

As an idea: If we don't want to clutter menu options, a better way to do this would be to not provide submenus: instead, just pop up an input box where you enter the character to be used for alignment in each use. This would be fast and practical, since in most cases it will be a single keystroke.

oBFusCATed:
Another problem is that the Aligner menu is missing from the main menu.

carra:
Yeah, it is certainly not there. I did not even know that it was supposed to be there!
Well it seems to me much quicker to use the context menu, but it won't hurt if the option is in the main menu as well.

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version