Skip to content
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

Merged
merged 16 commits into from
Aug 3, 2016

Conversation

thatkookooguy
Copy link
Member

@thatkookooguy thatkookooguy commented Jul 20, 2016

Change Summary

  • add some more built-in ace.js integrations to the menuBar component [resolves add more ace.js integrations #169]

    Basically, added

    • To Lower Case (see next mission)
    • To Upper Case (see next mission)
    • Select All (in Edit menu)
    • Find & Replace (in Edit menu)
    • Toggle Comment (see next mission)
    • Toggle Block Comment (see next mission)
    • Sort Lines (see next mission)
  • Made new menuBar menu called Selection [resolves add selection menu to the menuBar component #168]

    When text is selected, this menu will appear. based on your selection, some options will appear:

    Appear always:

    • UPPERCASE
    • LOWERCASE

    Appears when selecting a text case

    • camelCase
    • Title Case
    • kebab-case
    • snake_case

    Appears when selecting multiple lines

    • Sort Lines
    • Toggle Comment
    • Toggle Block Comment
  • 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

removed things left from copypasting in file▶️new + made print work.
Now, the only button that doesn't work in the menu (besides paste, but that's on purpose) is the share button 😃

  • did you link this PR to an issue?
  • did you lint your changes to both javascript and scss?
  • "I'm pretty sure I'll be able to read and understand this PR, even if I wasn't the author." - _said the PR author_

+ some case conversation buttons
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 20, 2016 23:41 Inactive
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) {

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

@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 20, 2016 23:42 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 20, 2016 23:43 Inactive
@thatkookooguy thatkookooguy self-assigned this Jul 20, 2016
@thatkookooguy thatkookooguy changed the title [DO NOT SUBMIT] add some more ace integrations [FEATURE] add some more ace integrations Jul 20, 2016
@thatkookooguy thatkookooguy changed the title [FEATURE] add some more ace integrations [FEATURE] Add some pizzazz to our menu bar component Jul 20, 2016
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 21, 2016 08:01 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 21, 2016 09:30 Inactive
* This message is:
*
* "Automatically scrolling cursor into view after selection change
* this will be disabled in the next version

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!
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 21, 2016 09:31 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 21, 2016 17:13 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 22, 2016 08:15 Inactive
var editor = vm.settings.currentEditor;
var selectionText = editor.getSelectedText();
var selection = vm.settings.currentEditor.selection.getRange();
selectionText = selectionText

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing semicolon after statement

@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 22, 2016 08:31 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 22, 2016 08:36 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 22, 2016 08:37 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 July 22, 2016 10:40 Inactive
@thatkookooguy thatkookooguy temporarily deployed to kibibit-demo-pr-167 August 1, 2016 14:20 Inactive
@thatkookooguy
Copy link
Member Author

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants