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

Implemented Send Message tab #27

Merged
merged 13 commits into from
Nov 27, 2018

Conversation

Techzune
Copy link
Contributor

Added an alternative way for the user to send messages (with backspace!) via serial. Instead of sending individual characters via console, this tab will send the entire message at once.
The send automatically will append a newline (\n) character to the message, but that can be overridden by holding down CTRL and sending.

@thirtythreeforty
Copy link
Owner

Oh hey, this is cool. Thanks for implementing this! Sorry, I've been out of pocket for the last few days.

Don't worry about the failing CI, it's busted on master and fixed in a branch I've been working on off and on.

Copy link
Owner

@thirtythreeforty thirtythreeforty left a comment

Choose a reason for hiding this comment

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

Last nitpick: the codebase uses tabs throughout. Can you make your changes match? Thanks again, this will be nice. I'm working on a couple other features to push out soon. An update is overdue.

src/MainWindow.cpp Outdated Show resolved Hide resolved
src/MainWindow.cpp Outdated Show resolved Hide resolved
src/mainwindow.ui Outdated Show resolved Hide resolved
src/mainwindow.ui Outdated Show resolved Hide resolved
@Techzune
Copy link
Contributor Author

GitHub Wiki pages may help explain software more without crowding the readme

@thirtythreeforty
Copy link
Owner

Great, thanks! I'm going to clean up whitespace (the rule is actually "tabs for indent, spaces for alignment" and I didn't expect your editor to change all the lines in the file, but nbd) and maybe tweak wording, and it should be ready to roll.

@Techzune
Copy link
Contributor Author

Woohoo! And I'm sorry about changing all the indentations! I'm used to JetBrains' IDE style, so I didn't realize that Qt Creator changed every line, lol.

@thirtythreeforty thirtythreeforty merged commit ecba8a0 into thirtythreeforty:master Nov 27, 2018
@thirtythreeforty

This comment has been minimized.

@Techzune Techzune deleted the send-tab branch November 27, 2018 16:42
Techzune pushed a commit to Techzune/bullycpp that referenced this pull request Nov 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants