Details
-
Bug
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
2.19.0
-
None
Description
The "Send message" screen has room for a few (sometimes minor) improvements. I've bundled them in one PR.
- (fix) fixed delete-header-item on page send-message-to-queue due to broken js code behind the delete-button (caught by viewing the diff between addressSendMessage.js and sendMessage.js);
- (fix) fixed format-message on page send-message-to-queue due to broken js code behind the format-button (this initially led me to believe that the button would apply syntax-color only, until I saw the intended behaviour on page send-message-to-address).
all of the below change apply to both send-message-to-queue and send-message-to-address pages:
- (smallfix) fixed small spelling error in a help-text where "in" was spelled as "ion";
- (smallfix) fixed small spelling error in a help-text where "it's" was spelled as "its";
- (cosmetic) the dropdown-box for the format selection has been shrunk and the button placed on the same line;
- (cosmetic) renamed button from "Send message" to "Send Message", just because the "Add Header" button on the same page also uses capitals.
I read that "CodeEditor" also supported YAML and TOML. But when I tried it out, the syntax colours only worked for YAML and the reformatting did not work for any of these two. so that part was dropped before it was even committed.
note: internally there are 2 "Send message" screens. one to send a message to an address, the other one to send a message to a queue. these screen look very similar and have the same title and button name. the 2 screens can be only be distinguished visually by looking at the help text from the 
button just after the page title. of course the one page is used when an address is selected in the tree; the other when a queue is selected in the tree.
Attachments
Attachments
Issue Links
- links to
