Author Topic: Events (Read 19419 times)

oBFusCATed · « **Reply #15 on:** November 10, 2011, 10:57:27 pm »

Quote from: Ceniza on November 10, 2011, 12:12:33 pm

It also allows you to comment/uncomment by selecting something and typing either / or *.

This is pretty annoying feature. 50% of the time it was deleting my selection.
The C::B solution is definitely better here:
Ctrl+Shift+x = comment the seletion
Ctrl+Shift+c = uncomment the selection
Works 100% of the time and it is pretty predictable.

Quote from: smallB on November 10, 2011, 12:08:37 pm

But "full configurability" should be a part of every feature not just this one.

This is definitely the wrong philosophy. Features should just work. Because it makes life easier.
We as developers should strive for this. Making it configurable is the easiest thing, most of the time, but not the best.

smallB · « **Reply #16 on:** November 11, 2011, 07:58:02 am »

@Obfuscated I strongly disagree with you on that one. Providing configurability of a feature allows every one to set it in a way that work best for them - in the spirit "What's good for you it doesn't mean it is good for me, and what you think is pretty to me seems awful". What you're saying is that there is one and correct/best way to suit everyone's needs and I'm saying that this isn't true and bad design practice.Just look around you, on most popular, successful software and think about it. The best way is to let everyone set the feature so it works the way suits the particular individual best, not the designer.
And at the end just to prove that what I'm saying is correct: you're saying something like:
"This is pretty annoying feature. 50% of the time it was deleting my selection." - It is not annoying to me. I love it. If I want to delete something I press either spacebar or backspace.
So you see, what's annoying for a one person may be of great help and usefulness for the other. There is no One Size Fits All. This approach is bad, and it doesn't work.

oBFusCATed · « **Reply #17 on:** November 11, 2011, 10:17:02 am »

But you've not understood what I've said.
There are plenty of approaches to implement a feature.
Some of the approaches, like the auto-commenting feature of V-Assists aren't robust and annoys some people.
The strive should be for non-annoying implementations of the features, like the manual commenting/uncommenting in C::B.

And I don't like to have to setup my C::B for 30 minutes every time I install new OS.
Most of the things I do is disable this, this, this and this.

smallB · « **Reply #18 on:** November 11, 2011, 10:55:31 am »

Quote from: oBFusCATed on November 11, 2011, 10:17:02 am

The strive should be for non-annoying implementations of the features, like the manual commenting/uncommenting in C::B.

But that's my point exactly - just because you think it is annoying or it indeed annoys you - fair enough - everyone has their own preferences, and likings, and here is the crux - everyone's preferences should be respected - not just yours. Because it is not annoying to me and I really like it. So what I'm saying is that everyone, you, me and other users should be able to set this feature the way suits them best. So it also means that you should be able to disable this if it does annoy you. It wouldn't be fair on you to not allow you to disable this, right? So why do you think it's fair on others not to be able to enable this? It's two way street. And that's good.

smallB · « **Reply #19 on:** November 11, 2011, 10:56:27 am »

Quote from: oBFusCATed on November 11, 2011, 10:17:02 am

And I don't like to have to setup my C::B for 30 minutes every time I install new OS.
Most of the things I do is disable this, this, this and this.

I don't think anyone likes it. But, but, but, but:
1. You can always export cb's settings and after installing new OS you can simply import them

.
2. How would you feel about it if you were robed from the privilege of disabling this, this and this? Because designer decided that disabling this is so annoying and it shouldn't be allowed. Would you agree with his approach/way of thinking?
3. There should be (and I'm going to add it as a my suggestion) option which would allow to choose (again you see? it should allow, to accommodate user like you - minimalist) how cb is set up.
What I mean by this is that there should be some kind of levels defined (which could also be edited) so for example user like you would pick:
Expert
someone else would pick:
Intermediate
and I would pick:
Novice
where everything would be set up as to ease my work as much as possible with this IDE. Default compiler, safest settings in it, every possible help when working with editor etc. etc etc.
But again, all this should be configurable so I with time with change my novice profile, and let's say remove some settings from compiler - or add them, but it still be my Novice profile.

Alpha · « **Reply #20 on:** November 11, 2011, 10:31:57 pm »

Here is an initial patch to show functionality. It currently has no settings; pressing any of:
' " ( ) [ ] < > { }
will trigger it (the final position of the caret is determined by the direction of the brace).

Currently, the altered text is deselected after activating. Does anyone like the idea of leaving " selected (but not the others) so that, for example, a user can select something and type:
"(wxT
to wrap it like:
wxT("sample text")

When creating a string with ", would it be expected behavior to escape any double quotes in between, or should I leave it as it currently is?

Because of the way wxWidgets implements the EVT_KEY_DOWN event, I am fairly certain it will not function properly on other keyboard layouts. I am working on a method of portablizing this.

When I get to creating the option of choosing between using keyboard shortcuts or the current method of activation (and, of course, disabling it completely), is there of set of six shortcuts (it could be fewer if only certain features are wanted) available that works on all keyboard layouts?

Code

Index: src/sdk/cbstyledtextctrl.cpp
===================================================================
--- src/sdk/cbstyledtextctrl.cpp	(revision 7568)
+++ src/sdk/cbstyledtextctrl.cpp	(working copy)
@@ -123,6 +123,148 @@
 
 void cbStyledTextCtrl::OnKeyDown(wxKeyEvent& event)
 {
+    wxString selectedText = GetSelectedText();
+    if (selectedText != wxEmptyString && !(event.AltDown() || event.ControlDown()))
+    {
+        switch (event.GetKeyCode())
+        {
+            case _T('\''):  // ' "
+            {
+                BeginUndoAction();
+                DeleteBack();
+                if (event.ShiftDown())
+                {
+                    InsertText(GetCurrentPos(), wxT("\"") + selectedText + wxT("\""));
+                }
+                else
+                {
+                    InsertText(GetCurrentPos(), wxT("'") + selectedText + wxT("'"));
+                }
+                SetEmptySelection(GetCurrentPos() + selectedText.Length() + 2);
+                EndUndoAction();
+                return;
+            }
+#ifdef __WXMSW__
+            case _T('9'):   // ( for wxMSW
+            {
+                if (!event.ShiftDown())
+                {
+                    break;
+                }
+            }
+#else
+            case _T('('):   // ( for wxGTK
+#endif
+            {
+                BeginUndoAction();
+                DeleteBack();
+                InsertText(GetCurrentPos(), wxT("(") + selectedText + wxT(")"));
+                EndUndoAction();
+                return;
+            }
+#ifdef __WXMSW__
+            case _T('0'):   // ) for wxMSW
+            {
+                if (!event.ShiftDown())
+                {
+                    break;
+                }
+            }
+#else
+            case _T(')'):   // ) for wxGTK
+#endif
+            {
+                BeginUndoAction();
+                DeleteBack();
+                InsertText(GetCurrentPos(), wxT("(") + selectedText + wxT(")"));
+                SetEmptySelection(GetCurrentPos() + selectedText.Length() + 2);
+                EndUndoAction();
+                return;
+            }
+            case _T(','):   // <
+            {
+                if (!event.ShiftDown())
+                {
+                    break;
+                }
+                BeginUndoAction();
+                DeleteBack();
+                InsertText(GetCurrentPos(), wxT("<") + selectedText + wxT(">"));
+                EndUndoAction();
+                return;
+            }
+            case _T('.'):   // >
+            {
+                if (!event.ShiftDown())
+                {
+                    break;
+                }
+                BeginUndoAction();
+                DeleteBack();
+                InsertText(GetCurrentPos(), wxT("<") + selectedText + wxT(">"));
+                SetEmptySelection(GetCurrentPos() + selectedText.Length() + 2);
+                EndUndoAction();
+                return;
+            }
+            case _T('['):   // [ {
+            {
+                BeginUndoAction();
+                if (event.ShiftDown())
+                {
+                    int startLine = LineFromPosition(GetSelectionStart());
+                    int endLine = LineFromPosition(GetSelectionEnd());
+                    if(startLine == endLine)
+                    {
+                        Home();
+                    }
+                    Tab();
+                    SetEmptySelection(GetLineEndPosition(endLine));
+                    NewLine();
+                    BackTab();
+                    InsertText(GetCurrentPos(), wxT("}"));
+                    SetEmptySelection(GetLineEndPosition(startLine - 1));
+                    NewLine();
+                    InsertText(GetCurrentPos(), wxT("{"));
+                }
+                else
+                {
+                    DeleteBack();
+                    InsertText(GetCurrentPos(), wxT("[") + selectedText + wxT("]"));
+                }
+                EndUndoAction();
+                return;
+            }
+            case _T(']'):   // ] }
+            {
+                BeginUndoAction();
+                if (event.ShiftDown())
+                {
+                    int startLine = LineFromPosition(GetSelectionStart());
+                    int endLine = LineFromPosition(GetSelectionEnd());
+                    if(startLine == endLine)
+                    {
+                        Home();
+                    }
+                    Tab();
+                    SetEmptySelection(GetLineEndPosition(startLine - 1));
+                    NewLine();
+                    InsertText(GetCurrentPos(), wxT("{"));
+                    SetEmptySelection(GetLineEndPosition(endLine + 1));
+                    NewLine();
+                    BackTab();
+                    InsertText(GetCurrentPos(), wxT("}"));
+                }
+                else
+                {
+                    DeleteBack();
+                    InsertText(GetCurrentPos(), wxT("[") + selectedText + wxT("]"));
+                    SetEmptySelection(GetCurrentPos() + selectedText.Length() + 2);
+                }
+                EndUndoAction();
+                return;
+            }
+        }
+    }
     switch (event.GetKeyCode())
     {
         case WXK_TAB:

Awaiting reports of functionality (or failure)...

Alpha · « **Reply #21 on:** November 12, 2011, 12:39:14 am »

Changes in this patch:

Works regardless of keyboard layout (I think)
Undo is now two steps; the first step will revert it to what would normally happen (without this patch's modification)
When creating a string, the relevant symbols are escaped
Strings remain selected

Code

Index: src/sdk/cbeditor.cpp
===================================================================
--- src/sdk/cbeditor.cpp	(revision 7582)
+++ src/sdk/cbeditor.cpp	(working copy)
@@ -3022,6 +3022,134 @@
     static int autoUnIndentValue = -1;
     static int autoUnIndentLine = -1;
 
+    if (!control->GetLastSelectedText().IsEmpty())
+    {
+        wxString selectedText = control->GetLastSelectedText();
+        switch (ch)
+        {
+            case _T('\''):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                selectedText.Replace(wxT("\\'"), wxT("'"));
+                selectedText.Replace(wxT("'"), wxT("\\'"));
+                control->InsertText(pos - 1, wxT("'") + selectedText + wxT("'"));
+                control->SetEmptySelection(pos + selectedText.Length() + 1);
+                control->EndUndoAction();
+                return;
+            }
+            case _T('"'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                selectedText.Replace(wxT("\\\""), wxT("\""));
+                selectedText.Replace(wxT("\""), wxT("\\\""));
+                control->InsertText(pos - 1, wxT("\"") + selectedText + wxT("\""));
+                control->SetCurrentPos(pos + selectedText.Length() + 1);
+                control->EndUndoAction();
+                return;
+            }
+            case _T('('):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("(") + selectedText + wxT(")"));
+                control->EndUndoAction();
+                return;
+            }
+            case _T(')'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("(") + selectedText + wxT(")"));
+                control->SetEmptySelection(pos + selectedText.Length() + 1);
+                control->EndUndoAction();
+                return;
+            }
+            case _T('['):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("[") + selectedText + wxT("]"));
+                control->EndUndoAction();
+                return;
+            }
+            case _T(']'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("[") + selectedText + wxT("]"));
+                control->SetEmptySelection(pos + selectedText.Length() + 1);
+                control->EndUndoAction();
+                return;
+            }
+            case _T('<'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("<") + selectedText + wxT(">"));
+                control->EndUndoAction();
+                return;
+            }
+            case _T('>'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("<") + selectedText + wxT(">"));
+                control->SetEmptySelection(pos + selectedText.Length() + 1);
+                control->EndUndoAction();
+                return;
+            }
+            case _T('{'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, selectedText);
+                control->SetSelectionVoid(pos - 1, pos + selectedText.Length() - 1);
+                int startLine = control->LineFromPosition(control->GetSelectionStart());
+                int endLine = control->LineFromPosition(control->GetSelectionEnd());
+                if(startLine == endLine)
+                {
+                    control->Home();
+                }
+                control->Tab();
+                control->SetEmptySelection(control->GetLineEndPosition(endLine));
+                control->NewLine();
+                control->BackTab();
+                control->InsertText(control->GetCurrentPos(), wxT("}"));
+                control->SetEmptySelection(control->GetLineEndPosition(startLine - 1));
+                control->NewLine();
+                control->InsertText(control->GetCurrentPos(), wxT("{"));
+                control->EndUndoAction();
+                return;
+            }
+            case _T('}'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, selectedText);
+                control->SetSelectionVoid(pos - 1, pos + selectedText.Length() - 1);
+                int startLine = control->LineFromPosition(control->GetSelectionStart());
+                int endLine = control->LineFromPosition(control->GetSelectionEnd());
+                if(startLine == endLine)
+                {
+                    control->Home();
+                }
+                control->Tab();
+                control->SetEmptySelection(control->GetLineEndPosition(startLine - 1));
+                control->NewLine();
+                control->InsertText(control->GetCurrentPos(), wxT("{"));
+                control->SetEmptySelection(control->GetLineEndPosition(endLine + 1));
+                control->NewLine();
+                control->BackTab();
+                control->InsertText(control->GetCurrentPos(), wxT("}"));
+                control->CharRight();
+                control->EndUndoAction();
+                return;
+            }
+        }
+    }
+
     // indent
     if (ch == _T('\n'))
     {
Index: src/sdk/cbstyledtextctrl.cpp
===================================================================
--- src/sdk/cbstyledtextctrl.cpp	(revision 7582)
+++ src/sdk/cbstyledtextctrl.cpp	(working copy)
@@ -123,6 +123,8 @@
 
 void cbStyledTextCtrl::OnKeyDown(wxKeyEvent& event)
 {
+    m_lastSelectedText = GetSelectedText();
+
     switch (event.GetKeyCode())
     {
         case WXK_TAB:
Index: src/include/cbstyledtextctrl.h
===================================================================
--- src/include/cbstyledtextctrl.h	(revision 7582)
+++ src/include/cbstyledtextctrl.h	(working copy)
@@ -22,6 +22,7 @@
         cbStyledTextCtrl(wxWindow* pParent, int id, const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize, long style = 0);
         virtual ~cbStyledTextCtrl();
         wxDateTime GetLastFocusTime() const {return m_lastFocusTime;}
+        wxString GetLastSelectedText() const {return m_lastSelectedText;}
 
         void EnableTabSmartJump(bool enable = true);
         bool IsCharacter(int style);
@@ -52,6 +53,7 @@
         int m_bracePosition;
         int m_lastPosition;
         bool m_tabSmartJump;
+        wxString m_lastSelectedText;
 
         static std::map<int, std::set<int> > CharacterLexerStyles, StringLexerStyles, PreprocessorLexerStyles, CommentLexerStyles;

Alpha · « **Reply #22 on:** November 13, 2011, 05:07:56 am »

Changes in this patch:

Activation is now controlled by an option (Settings->Editor... "Selection brace completion")
Removed a bit of redundant code

Code

Index: src/sdk/editorconfigurationdlg.cpp
===================================================================
--- src/sdk/editorconfigurationdlg.cpp	(revision 7582)
+++ src/sdk/editorconfigurationdlg.cpp	(working copy)
@@ -20,7 +20,7 @@
     #include <wx/stattext.h>
     #include <wx/textdlg.h>
     #include <wx/xrc/xmlres.h>
-    
+
     #include "manager.h"
     #include "configmanager.h"
     #include "pluginmanager.h"
@@ -118,6 +118,7 @@
     XRCCTRL(*this, "chkUseChangebar", wxCheckBox)->SetValue(m_EnableChangebar);
     XRCCTRL(*this, "chkShowIndentGuides", wxCheckBox)->SetValue(cfg->ReadBool(_T("/show_indent_guides"), false));
     XRCCTRL(*this, "chkBraceSmartIndent", wxCheckBox)->SetValue(cfg->ReadBool(_T("/brace_smart_indent"), true));
+    XRCCTRL(*this, "chkSelectionBraceCompletion", wxCheckBox)->SetValue(cfg->ReadBool(_T("/selection_brace_completion"), false));
     XRCCTRL(*this, "chkTabIndents", wxCheckBox)->SetValue(cfg->ReadBool(_T("/tab_indents"), true));
     XRCCTRL(*this, "chkBackspaceUnindents", wxCheckBox)->SetValue(cfg->ReadBool(_T("/backspace_unindents"), true));
     XRCCTRL(*this, "chkWordWrap", wxCheckBox)->SetValue(cfg->ReadBool(_T("/word_wrap"), false));
@@ -795,6 +796,7 @@
         cfg->Write(_T("/use_tab"),                             XRCCTRL(*this, "chkUseTab", wxCheckBox)->GetValue());
         cfg->Write(_T("/show_indent_guides"),                  XRCCTRL(*this, "chkShowIndentGuides", wxCheckBox)->GetValue());
         cfg->Write(_T("/brace_smart_indent"),                  XRCCTRL(*this, "chkBraceSmartIndent", wxCheckBox)->GetValue());
+        cfg->Write(_T("/selection_brace_completion"),          XRCCTRL(*this, "chkSelectionBraceCompletion", wxCheckBox)->GetValue());
         cfg->Write(_T("/tab_indents"),                         XRCCTRL(*this, "chkTabIndents", wxCheckBox)->GetValue());
         cfg->Write(_T("/backspace_unindents"),                 XRCCTRL(*this, "chkBackspaceUnindents", wxCheckBox)->GetValue());
         cfg->Write(_T("/word_wrap"),                           XRCCTRL(*this, "chkWordWrap", wxCheckBox)->GetValue());
Index: src/sdk/cbeditor.cpp
===================================================================
--- src/sdk/cbeditor.cpp	(revision 7582)
+++ src/sdk/cbeditor.cpp	(working copy)
@@ -2897,7 +2897,7 @@
             wxTreeItemId sel = Manager::Get()->GetProjectManager()->GetTreeSelection();
             if (sel.IsOk())
                 tree->SelectItem(sel, false);
-            
+
             const wxTreeItemId &itemId = m_pProjectFile->GetTreeItemId();
             if (itemId.IsOk())
             {
@@ -3022,6 +3022,104 @@
     static int autoUnIndentValue = -1;
     static int autoUnIndentLine = -1;
 
+    bool SelectionBraceCompletion = Manager::Get()->GetConfigManager(_T("editor"))->ReadBool(_T("/selection_brace_completion"), false);
+    if (SelectionBraceCompletion && !control->GetLastSelectedText().IsEmpty())
+    {
+        wxString selectedText = control->GetLastSelectedText();
+        switch (ch)
+        {
+            case _T('\''):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                selectedText.Replace(wxT("\\'"), wxT("'"));
+                selectedText.Replace(wxT("'"), wxT("\\'"));
+                control->InsertText(pos - 1, wxT("'") + selectedText + wxT("'"));
+                control->SetEmptySelection(pos + selectedText.Length() + 1);
+                control->EndUndoAction();
+                return;
+            }
+            case _T('"'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                selectedText.Replace(wxT("\\\""), wxT("\""));
+                selectedText.Replace(wxT("\""), wxT("\\\""));
+                control->InsertText(pos - 1, wxT("\"") + selectedText + wxT("\""));
+                control->SetCurrentPos(pos + selectedText.Length() + 1);
+                control->EndUndoAction();
+                return;
+            }
+            case _T('('):
+            case _T(')'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("(") + selectedText + wxT(")"));
+                if(ch == _T(')'))
+                {
+                    control->SetEmptySelection(pos + selectedText.Length() + 1);
+                }
+                control->EndUndoAction();
+                return;
+            }
+            case _T('['):
+            case _T(']'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("[") + selectedText + wxT("]"));
+                if(ch == _T(']'))
+                {
+                    control->SetEmptySelection(pos + selectedText.Length() + 1);
+                }
+                control->EndUndoAction();
+                return;
+            }
+            case _T('<'):
+            case _T('>'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, wxT("<") + selectedText + wxT(">"));
+                if(ch == _T('>'))
+                {
+                    control->SetEmptySelection(pos + selectedText.Length() + 1);
+                }
+                control->EndUndoAction();
+                return;
+            }
+            case _T('{'):
+            case _T('}'):
+            {
+                control->BeginUndoAction();
+                control->DeleteBack();
+                control->InsertText(pos - 1, selectedText);
+                control->SetSelectionVoid(pos - 1, pos + selectedText.Length() - 1);
+                int startLine = control->LineFromPosition(control->GetSelectionStart());
+                int endLine = control->LineFromPosition(control->GetSelectionEnd());
+                if(startLine == endLine)
+                {
+                    control->Home();
+                }
+                control->Tab();
+                control->SetEmptySelection(control->GetLineEndPosition(endLine));
+                control->NewLine();
+                control->BackTab();
+                control->InsertText(control->GetCurrentPos(), wxT("}"));
+                control->SetEmptySelection(control->GetLineEndPosition(startLine - 1));
+                control->NewLine();
+                control->InsertText(control->GetCurrentPos(), wxT("{"));
+                if(ch == _T('}'))
+                {
+                    control->SetEmptySelection(control->GetLineEndPosition(endLine + 2));
+                }
+                control->EndUndoAction();
+                return;
+            }
+        }
+    }
+
     // indent
     if (ch == _T('\n'))
     {
Index: src/sdk/cbstyledtextctrl.cpp
===================================================================
--- src/sdk/cbstyledtextctrl.cpp	(revision 7582)
+++ src/sdk/cbstyledtextctrl.cpp	(working copy)
@@ -123,6 +123,8 @@
 
 void cbStyledTextCtrl::OnKeyDown(wxKeyEvent& event)
 {
+    m_lastSelectedText = GetSelectedText();
+
     switch (event.GetKeyCode())
     {
         case WXK_TAB:
Index: src/sdk/resources/editor_configuration.xrc
===================================================================
--- src/sdk/resources/editor_configuration.xrc	(revision 7582)
+++ src/sdk/resources/editor_configuration.xrc	(working copy)
@@ -154,7 +154,7 @@
 															</object>
 															<object class="sizeritem">
 																<object class="wxStaticBoxSizer">
-																	<label>End-of-line options</label>
+																	<label>Highlight occurrences</label>
 																	<orient>wxVERTICAL</orient>
 																	<object class="sizeritem">
 																		<object class="wxFlexGridSizer">
@@ -162,64 +162,54 @@
 																			<vgap>4</vgap>
 																			<hgap>4</hgap>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkShowEOL">
-																					<label>Show end-of-line chars</label>
+																				<object class="wxCheckBox" name="chkHighlightOccurrences">
+																					<label>Highlight occurrences</label>
+																					<checked>1</checked>
 																				</object>
 																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																			</object>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkStripTrailings">
-																					<label>Strip trailing blanks</label>
+																				<object class="wxCheckBox" name="chkHighlightOccurrencesCaseSensitive">
+																					<label>Case sensitive</label>
 																					<checked>1</checked>
 																				</object>
-																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
+																				<flag>wxLEFT|wxALIGN_LEFT|wxALIGN_TOP</flag>
+																				<border>10</border>
 																			</object>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkEnsureFinalEOL">
-																					<label>End files with blank line</label>
+																				<object class="wxCheckBox" name="chkHighlightOccurrencesWholeWord">
+																					<label>Whole words only</label>
+																					<checked>1</checked>
 																				</object>
-																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
+																				<flag>wxLEFT|wxALIGN_LEFT|wxALIGN_TOP</flag>
+																				<border>10</border>
 																			</object>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkEnsureConsistentEOL">
-																					<label>Ensure consistent EOLs</label>
-																				</object>
-																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
-																			</object>
-																			<object class="sizeritem">
 																				<object class="wxBoxSizer">
 																					<object class="sizeritem">
-																						<object class="wxStaticText" name="ID_STATICTEXT2">
-																							<label>End-of-line mode:</label>
+																						<object class="wxStaticText" name="stHighlightColour">
+																							<label>Highlight colour</label>
 																						</object>
-																						<flag>wxTOP|wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
-																						<border>4</border>
+																						<flag>wxRIGHT|wxALIGN_LEFT|wxALIGN_CENTER_VERTICAL</flag>
+																						<border>5</border>
 																					</object>
 																					<object class="sizeritem">
-																						<object class="wxComboBox" name="cmbEOLMode">
-																							<content>
-																								<item>CR LF</item>
-																								<item>CR</item>
-																								<item>LF</item>
-																							</content>
-																							<style>wxCB_READONLY</style>
+																						<object class="wxButton" name="btnHighlightColour">
+																							<label>...</label>
+																							<bg>#FF0000</bg>
 																						</object>
-																						<flag>wxLEFT|wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
-																						<border>16</border>
-																						<option>1</option>
+																						<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																					</object>
 																				</object>
-																				<flag>wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
-																				<option>1</option>
+																				<flag>wxLEFT|wxALIGN_LEFT|wxALIGN_TOP</flag>
+																				<border>10</border>
 																			</object>
 																		</object>
 																		<flag>wxALL|wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
 																		<border>4</border>
-																		<option>1</option>
 																	</object>
 																</object>
 																<flag>wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
-																<border>4</border>
 																<option>1</option>
 															</object>
 															<object class="sizeritem">
@@ -260,17 +250,23 @@
 																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																			</object>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkShowIndentGuides">
-																					<label>Show indentation guides</label>
+																				<object class="wxCheckBox" name="chkBraceSmartIndent">
+																					<label>Brace Smart Indent</label>
 																				</object>
 																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																			</object>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkBraceSmartIndent">
-																					<label>Brace Smart Indent</label>
+																				<object class="wxCheckBox" name="chkSelectionBraceCompletion">
+																					<label>Selection brace completion</label>
 																				</object>
 																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																			</object>
+																			<object class="sizeritem">
+																				<object class="wxCheckBox" name="chkShowIndentGuides">
+																					<label>Show indentation guides</label>
+																				</object>
+																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
+																			</object>
 																		</object>
 																		<flag>wxALL|wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
 																		<border>4</border>
@@ -281,7 +277,7 @@
 															</object>
 															<object class="sizeritem">
 																<object class="wxStaticBoxSizer">
-																	<label>Highlight occurrences</label>
+																	<label>End-of-line options</label>
 																	<orient>wxVERTICAL</orient>
 																	<object class="sizeritem">
 																		<object class="wxFlexGridSizer">
@@ -289,54 +285,64 @@
 																			<vgap>4</vgap>
 																			<hgap>4</hgap>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkHighlightOccurrences">
-																					<label>Highlight occurrences</label>
-																					<checked>1</checked>
+																				<object class="wxCheckBox" name="chkShowEOL">
+																					<label>Show end-of-line chars</label>
 																				</object>
 																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																			</object>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkHighlightOccurrencesCaseSensitive">
-																					<label>Case sensitive</label>
+																				<object class="wxCheckBox" name="chkStripTrailings">
+																					<label>Strip trailing blanks</label>
 																					<checked>1</checked>
 																				</object>
-																				<flag>wxLEFT|wxALIGN_LEFT|wxALIGN_TOP</flag>
-																				<border>10</border>
+																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																			</object>
 																			<object class="sizeritem">
-																				<object class="wxCheckBox" name="chkHighlightOccurrencesWholeWord">
-																					<label>Whole words only</label>
-																					<checked>1</checked>
+																				<object class="wxCheckBox" name="chkEnsureFinalEOL">
+																					<label>End files with blank line</label>
 																				</object>
-																				<flag>wxLEFT|wxALIGN_LEFT|wxALIGN_TOP</flag>
-																				<border>10</border>
+																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
 																			</object>
 																			<object class="sizeritem">
+																				<object class="wxCheckBox" name="chkEnsureConsistentEOL">
+																					<label>Ensure consistent EOLs</label>
+																				</object>
+																				<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
+																			</object>
+																			<object class="sizeritem">
 																				<object class="wxBoxSizer">
 																					<object class="sizeritem">
-																						<object class="wxStaticText" name="stHighlightColour">
-																							<label>Highlight colour</label>
+																						<object class="wxStaticText" name="ID_STATICTEXT2">
+																							<label>End-of-line mode:</label>
 																						</object>
-																						<flag>wxRIGHT|wxALIGN_LEFT|wxALIGN_CENTER_VERTICAL</flag>
-																						<border>5</border>
+																						<flag>wxTOP|wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
+																						<border>4</border>
 																					</object>
 																					<object class="sizeritem">
-																						<object class="wxButton" name="btnHighlightColour">
-																							<label>...</label>
-																							<bg>#FF0000</bg>
+																						<object class="wxComboBox" name="cmbEOLMode">
+																							<content>
+																								<item>CR LF</item>
+																								<item>CR</item>
+																								<item>LF</item>
+																							</content>
+																							<style>wxCB_READONLY</style>
 																						</object>
-																						<flag>wxALIGN_LEFT|wxALIGN_TOP</flag>
+																						<flag>wxLEFT|wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
+																						<border>16</border>
+																						<option>1</option>
 																					</object>
 																				</object>
-																				<flag>wxLEFT|wxALIGN_LEFT|wxALIGN_TOP</flag>
-																				<border>10</border>
+																				<flag>wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
+																				<option>1</option>
 																			</object>
 																		</object>
 																		<flag>wxALL|wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
 																		<border>4</border>
+																		<option>1</option>
 																	</object>
 																</object>
 																<flag>wxEXPAND|wxALIGN_LEFT|wxALIGN_TOP</flag>
+																<border>4</border>
 																<option>1</option>
 															</object>
 														</object>

Alpha · « **Reply #23 on:** November 18, 2011, 12:17:16 am »

Out of curiosity, has anyone had the chance to try this?

Any comments or suggestions?

(Also, I am also still unsure of what keyboard shortcut(s) to (optionally) bind these functions to. Of the vi documentation I read (assuming I understood correctly), it would require a key combination that does not correspond to any normal keyboard shortcuts.)

MortenMacFly · « **Reply #24 on:** November 18, 2011, 07:45:29 am »

Quote from: Alpha on November 18, 2011, 12:17:16 am

Out of curiosity, has anyone had the chance to try this?

Its on my ToDo list, but I'm afraid I didn't find the time yet...

Alpha · « **Reply #25 on:** November 19, 2011, 03:08:47 am »

Quote from: MortenMacFly on November 18, 2011, 07:45:29 am

Its on my ToDo list, but I'm afraid I didn't find the time yet...

OK, that is fine.

Changes in this patch:

Keyboard shortcut Alt-I added; if the next character typed is one of ' " ( ) [ ] < > { }, it is treated the same as if "Selection brace completion" is checked
Removed a bit more of redundant code

Submitted to BerliOS as Patch #3234.

Code::Blocks Forums

News:

Author Topic: Events (Read 19419 times)

oBFusCATed

Re: Events

smallB

Re: Events

oBFusCATed

Re: Events

smallB

Re: Events

smallB

Re: Events

Alpha

Re: Events

Alpha

Re: Events

Alpha

Re: Events

Alpha

Re: Events

MortenMacFly

Re: Events

Alpha

Re: Events