-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEATURE] Add some pizzazz to our menu bar component #167
Conversation
+ some case conversation buttons
var selection = vm.settings.currentEditor.selection.getRange(); | ||
var titleCased = selectionText.replace( | ||
/[_-\s]([a-zA-Z])|([a-z])([A-Z])/g, | ||
function (g, singleLetter, firstLetter, secondLetter) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Illegal space before opening round brace
* This message is: | ||
* | ||
* "Automatically scrolling cursor into view after selection change | ||
* this will be disabled in the next version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Illegal trailing whitespace
good boy!
var editor = vm.settings.currentEditor; | ||
var selectionText = editor.getSelectedText(); | ||
var selection = vm.settings.currentEditor.selection.getRange(); | ||
selectionText = selectionText |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing semicolon after statement
@ortichon I'm merging this this evening (skipping your CR) since a long time have passed (12 days) and I want to continue to other things Talk to me if you planned getting to this soon and I'll wait some more. |
Change Summary
add some more built-in
ace.js
integrations to themenuBar
component [resolves add more ace.js integrations #169]Made new
menuBar
menu calledSelection
[resolves add selection menu to the menuBar component #168]Allow
ace.js
to scroll the last line to the top (like most modern code editors) [resolves Add scroll last line to top behaviour for the editor #170]Cleaned up more buttons in the menu bar