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

Conform user-interface text to conventions for trailing punctuation #2209

Merged
merged 12 commits into from
Jun 18, 2020

Conversation

brainchild0
Copy link
Contributor

@brainchild0 brainchild0 commented Jun 9, 2020

Following #2075, which revised grammar, the current request concerns use of punctuation in user strings, so that they follow conventions familiar for graphic interfaces.

Note that this request does include some further revision to grammar (d791e52, d69622d) not included revision to language not included in #2075.

The other changes are the following:

  • Remove full stops (.) trailing widget labels (f223088, 4cb011d).
  • End with ellipsis ('...') for all menu entries that open windows or other applications (c713a1c, 4cb011d).
  • Remove trailing colons (':'), which had appeared in the GitHub section of the image uploader section of the preferences window (c713a1c).

  • Bug fix?: no
  • New feature?: no
  • Breaking changes?: no
  • Deprecations?: no
  • New tests added?: not needed
  • Fixed tickets: n/a
  • License: MIT

- End with elipsis ('...') for all menu entries that open windows or
  other applications.

- Remove trailing colons (':'), which had appeared in the GitHub section
  of the image uploader section of the preferences window.
This was referenced Jun 9, 2020
Copy link
Contributor

@fxha fxha left a comment

Choose a reason for hiding this comment

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

Thanks for your suggestions. All buttons (el-button, button, ...) and entries in help.js shouldn't include ending ellipsis, even if a dialog or URL is opened and we should also keep the ending dots if not followed by :.

src/renderer/prefComponents/markdown/index.vue Outdated Show resolved Hide resolved
src/renderer/prefComponents/imageUploader/index.vue Outdated Show resolved Hide resolved
@brainchild0
Copy link
Contributor Author

brainchild0 commented Jun 11, 2020

I can restore the colons in the GitHub window, but then it would be inconsistent with other parts of the application and general conventions. What would be the reason for such a preference?

Keeping full stops in cases only when no colon follows is complicated because the colons are generated by the toolkit, not part of the application string. This observation suggests that the toolkit shares my the assumption that no full stop terminates a widget label, which is usually a word or short phrase rather than a full sentence. Based on the layout specification files, what rule would you suggest to distinguish which strings should end in full stop, and why would you suggest that any should have one?

@brainchild0
Copy link
Contributor Author

Conventions are mixed for help menus (see below). I'll revert help.js if it is the preference.


Screenshot from 2020-06-11 02-34-45
Screenshot from 2020-06-11 02-35-52

@fxha
Copy link
Contributor

fxha commented Jun 11, 2020

I can restore the colons in the GitHub window, but then it would be inconsistent with other parts of the application and general conventions. What would be the reason for such a preference?

There are colons because it's listed as text/input, the text refers directly to the text input below like in documents Name: <text field>.

Based on the layout specification files, what rule would you suggest to distinguish which strings should end in full stop, and why would you suggest that any should have one?

All strings that aren't on ranges (range elements) should end with a full stop because we end the phrase. I would say, we could go without if the element is design as <checkbox><text> with an optional description on top.

Conventions are mixed for help menus (see below). I'll revert help.js if it is the preference.

I'm unsure whether we should use ellipses on menu entries or not but we should remove it from buttons. Actually I'm fine with that because it indicate that the entry opens an external dialog, link etc.

@brainchild0
Copy link
Contributor Author

I see that many applications use colons for the labels, but do so for drop-down menus as well as text boxes. I would suggest keeping this consistent, that is, using them in both cases or neither. (I personally think that colons used in this way are unnecessary and distracting, but convention certainly supports their inclusion.)

Full stops are never used.

Ellipses are sometimes omitted in an application, but most mature, stable, quality applications use them for both menu items and buttons that open new windows. Some but not all application specifically omit them in the help menu.

Presently, the toolkit inserts colons in some cases, whereas in other cases they are given in the application strings. I strongly suggest that in all cases they are either inserted by the toolkit, if at all, or given in the strings. The mixed model is likely to generate confusion, diminish maintainability, and result in bugs.

@brainchild0
Copy link
Contributor Author

I have restored the colons, which we can keep until we have a clearer convention for when they would be used and what rule the toolkit has in displaying them.

I am reluctant to make the other changes without a more careful review.

@fxha
Copy link
Contributor

fxha commented Jun 16, 2020

I see that many applications use colons for the labels, but do so for drop-down menus as well as text boxes.

That may be true but we are only using colons if there is a value (such the font size) behind it. Then, you could add colons to all strings because it's always followed by a checkbox or other interactive element.

Ellipses are sometimes omitted in an application, but most mature, stable, quality applications use them for both menu items and buttons that open new windows. Some but not all application specifically omit them in the help menu.

I think this is pure preference and, for sure, it makes the menus cleaner and indicate external links. Please readd the ellipsis on menu items but not on buttons, sorry for the inconvenience.

Presently, the toolkit inserts colons in some cases, whereas in other cases they are given in the application strings. I strongly suggest that in all cases they are either inserted by the toolkit, if at all, or given in the strings. The mixed model is likely to generate confusion, diminish maintainability, and result in bugs.

I don't see any problems here as we only insert colons on ranges that indicate a value. We might refactor the range element later and move the colons to the string for better clarity and to be more consistent 👍 Please open an issue or change it on this PR for this if you want.

@brainchild0
Copy link
Contributor Author

  • But no colons for check box labels (which follow the box), but only for drop-down menus and text boxes (which precede the control)...
  • So should I keep 8dd5308 or reverse it (again)?
  • Regarding ellipses in the dialog boxes, yes, I agree it is preference. It is not wrong to omit them. But I find from inspection of applications on my system that more of the quality applications have them. Choosing to include them may make the application look higher quality and better planned. But I will remove them in this request, as you asked.

@brainchild0 brainchild0 marked this pull request as ready for review June 17, 2020 05:00
@fxha
Copy link
Contributor

fxha commented Jun 17, 2020

Please keep 8dd5308, add colons before drop-down menus (to be consistent with text boxes) and add ellipsis on menu entries (external links or dialog).

Regarding ellipses in the dialog boxes, yes, I agree it is preference. It is not wrong to omit them.

For now, we should keep the buttons without ellipsis. I think it's fine to add ellipses on setting button but then we have to include ellipses on editor buttons as well that look strange to me (e.g. side bar Open Folder).

@brainchild0
Copy link
Contributor Author

I agree about consistency with dialog boxes versus side bar, but I had added ellipsis in both. See the recent 7935b20, which removes them from both places (after I added them earlier).

The problem with adding colons for text boxes is that the toolkit is currently adding text after the label, which in many cases includes a colon. I can open a ticket to address these questions after this request is merged, but I think what you're asking is outside the scope of this change set.

@fxha
Copy link
Contributor

fxha commented Jun 17, 2020

The problem with adding colons for text boxes is that the toolkit is currently adding text after the label, which in many cases includes a colon.

If you talk about the linked image below, just change {{description}}: to {{description}} from this line and correct all strings on <range elements inside src/renderer/prefComponents. But I agree to create a separate PR for this.

@brainchild0
Copy link
Contributor Author

The objective of this request is to transition the use of punctuation toward standard uses for user interfaces. It is intended as a natural pairing with #2075 and targeted for the same release.

A fully satisfactory solution would depend on adopting a few clear conventions for the project, and likely making a few design changes to achieve them, including more consistent use of templates in the Vue.js pages. With HTML and style sheets, a range of options, including character styling and positioning, are available, in addition to string templates.

For now, I am hoping to include a clear, simple, safe, regression-free, and well-contained change set in the upcoming release.

Presumably the suggested change to the range template is intended to accompany changes to some but not all of the user strings. Instead, I prefer to find a proper solution after this request would be merged. I am happy to open a ticket at that time for discussion.

A very small change to the template that might be included in this request is represented in 03cfe8a. This change removes the colons from the range labels. Even if the colons are appropriate in the labels, I would not characterize this change as a regression because the colons should separate the labels from the controls, not the parts of the labels from one anther as they currently do.

For now, I suggest the following:

  1. Make any combination (all, some or none) of the following changes, as you prefer, to this request:
  2. Merge this request into develop.
  3. Merge or rebase into the planned release.

@brainchild0
Copy link
Contributor Author

brainchild0 commented Jun 18, 2020

I created an example of how to improve the templates in preferences box. (See screen captures and change set.)

Hopefully we can just push in this change set and then resolve how to proceed.

@fxha
Copy link
Contributor

fxha commented Jun 18, 2020

Please add ellipsis in help menu like in c2ab861 and include 03cfe8a if you want. Then, I can merge the PR later and release v0.16.2.

@brainchild0
Copy link
Contributor Author

brainchild0 commented Jun 18, 2020

I'm not excited about any template changes in this change set. I have restored the ellipses in the help menu.

Please merge as soon as the tests pass, and you are able to do so.

@fxha fxha merged commit b71b6ff into marktext:develop Jun 18, 2020
Jocs added a commit that referenced this pull request Jul 16, 2021
* update readme (#2008)

* update readme

* remove unused codes

* Update third party

* remove unused codes

* fix: style error in editor setting

* Release v0.16.1 (#2011)

* Fix typo (#2118)

* Include Chocolatey package installation option (#2107)

* Update dependencies (#2133)

* Fix max width of editor tabs (#2097)

* Fix max width of editor tabs

* Update sponsor (#2141)

* update: sponsors

* update img width and height

* Add ability to paste images to Features (#2127)

* Fix empty shortcuts in shortcut validator (#2130)

* Fix Hunspell dictionary downloader on Windows (#2131)

* Add missing encoding entry (#2132)

* Improve heading font size scaling (#2135)

* Reduce dependency bundle size (#2136)

* Fix Hunspell spell checker initialization on Windows (#2137)

* Disable and hide unnecessary settings options (#2138)

* Increase indentation for sidebar treeview (#2161)

* Fix image export issues on Windows (#2129)

* Fix footnote renderer exception in markedjs (#2162)

* Add GitLab math block support (#2119)

* update: yarn.lock file (#2178)

* feat: switch tabs using alt-#num (#1975)

* Fix bad font when exporting document on macOS (#2190)

* 📝 update zh_cn.md (#2192)

* Improve English grammar in user-interface text. (#2075)

* Fix unnecessary character sanitation on clipboard output (#2197)

* Fix double used shortcut (#2198)

* Fix double used shortcut

* Update documentation

* Update dependencies (#2199)

* Support setting text direction via comand palette (#2194) (#2195)

* Update bug issue template (#2215)

* Fix inline html renderer (#2224)

* Fix sub- and superscript in HTML output

* Update dependencies

* Conform user-interface text to conventions for trailing punctuation (#2209)

* Release v0.16.2 (#2200)

Bump version to 0.16.2 and update documentation.

* Replace PRC emoji for zh_TW translation with Taiwan flag (#2278)

* Update FAQ.md (#2290)

Clarify what aidou is for non-chinese users

* Fix markdown syntax document mistake for table content align (#2282) (#2296)

* Update pt.md (#2350)

* Update pt.md

My annual contribution to Mark Text, with its documentation accessible to all Brazilians and Portuguese.

* Update pt.md

* Update pt.md

* Update pt.md

* Update pt.md

* Update pt.md

* Update pt.md

* Update pt.md

* Update pt.md

* Update regex of auto link to fix trailing slashes (#2354)

* Update FAQ.md - link to Portable (#2337)

Fixing link to portable page.

* updated turkish translation of readme.md (#2366)

* pass file.id to closeTab on middle click (#2272)

* Add codeblock highlight and coloration for the SPARQL and Turtle languages (#2365)

* fix image paths on readme tr.md (#2374)

* Fix issue with not being able to copy table cell (#2375)

* fixed issue#2383 (#2397)

* Fix quick open bug and freeze constants to avoid accidental mutation (#2323)

* fix: #2330 enable shift+tab for table navigation (#2331)

* fix: prism language missing while load dependent language (#1752) (#2335)

* Fix graphite light wordcount (#2392)

* Update dependencies (minor only) (#2402)

* fix: typewriter shake after press enter (#1852)

* Support negative zoom (#2412)

* Restrict export dialog file extensions (#2408)

* Allow BCP-47 language codes in spell checker (#2410)

* Fix footnote inline code font size (#2411)

* feat: add shortcut for toggle-toc (#1863) (#2326)

* Make global menu fields mutable (#2407)

* Replace text when only one table cell is selected (#2413)

* Allow to copy table cells text as plaintext (#2409)

* Fix font selection autocomplete (#2420)

* Updated korean translation of README.md (#2432)

I updated ko.md (korean translation of README.md) to match the latest version of README.md

* Rebase markedjs bug fixes (#2419)

* Add option to disable HTML rendering (#2414)

* Fix symbolic links and related path issues (#2416)

* Upgrade to N-API and Electron 11 (#2422)

* Upgrade to N-API and Electron 11

* Update Electron and remove workaround

* Fix ESLint error

* Disble MathML due to DOMPurify security issues (#2436)

* Update Simplified Chinese translation of README (#2445)

* Improve undo history (#2437)

* Add TOC generator to document exporter (#2391)

* Add support for relative image directory (#2439)

* Fix installation instructions in README.md (#2449)

* Update README.md

Calling brew cask install is disabled

* Update i18n files

* Release v0.16.3 (#2448)

Co-authored-by: Ran Luo <ransixi@gmail.com>
Co-authored-by: saymoon <saymoon@gmail.com>
Co-authored-by: shaunmbg <shaunmbg@users.noreply.github.com>
Co-authored-by: Micael Dias <diasmicaelandre@gmail.com>
Co-authored-by: Dan Dascalescu <ddascalescu+github@gmail.com>
Co-authored-by: BeatLink <47278150+BeatLink@users.noreply.github.com>
Co-authored-by: He Linming <hlm52pk@163.com>
Co-authored-by: Royce <royce2003@qq.com>
Co-authored-by: brainchild0 <34869175+brainchild0@users.noreply.github.com>
Co-authored-by: Rasmus Eneman <rasmus@eneman.eu>
Co-authored-by: nyanpasu64 <nyanpasu64@tuta.io>
Co-authored-by: Sbelletier <sbelletier@gmail.com>
Co-authored-by: 小新 <xinping.wang@hotmail.com>
Co-authored-by: Enderson Menezes <endersonster@gmail.com>
Co-authored-by: Sweet Liquid <himejisyana@hotmail.com>
Co-authored-by: Nick Ali <1514992+nickali@users.noreply.github.com>
Co-authored-by: erentar <erentar2002@gmail.com>
Co-authored-by: Max Lantas <mnxndev@outlook.com>
Co-authored-by: Vincent Emonet <vincent.emonet@gmail.com>
Co-authored-by: Amauri Bizerra <amauriairesfilho@gmail.com>
Co-authored-by: Anton Freeman <asfreeman12@gmail.com>
Co-authored-by: Dan Munckton <munckfish@gmail.com>
Co-authored-by: evan-cohen <evancohen97@gmail.com>
Co-authored-by: zmen <sandoverflow@gmail.com>
Co-authored-by: bmvisoky <49735648+bmvisoky@users.noreply.github.com>
Co-authored-by: YeloMelon <42390003+YeloMelon@users.noreply.github.com>
Co-authored-by: MingZZX <41257880+mzzx@users.noreply.github.com>
Co-authored-by: カワリミ人形 <kawarimidoll+git@gmail.com>
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.

None yet

2 participants