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

Add NormalizeEnDashesFormatter #4045

Merged
merged 4 commits into from
May 25, 2018
Merged

Conversation

koppor
Copy link
Member

@koppor koppor commented May 21, 2018

Many titles are of the form Word Word - Subtitle. However, - is the wrong dash. It has to be en dash. It is argued that it has to be an em dash, but for me an em dash is too long and nearly all persons do not use it.

I discovered a comment at that writing.sx question:

The Oxford Style Guide actually recommends an en-dash flanked with two spaces over an em-dash and no spaces

new

PR builds on #4044


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@koppor koppor changed the base branch from feature/abstract-formatter to master May 24, 2018 14:56
Copy link
Member

@Siedlerchr Siedlerchr left a comment

Choose a reason for hiding this comment

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

If you now add the missing localization key, it's ready for merge

@koppor koppor force-pushed the feature/fix-dash-formatter branch from 7185874 to 33d1209 Compare May 25, 2018 08:09
@stefan-kolb stefan-kolb merged commit b029c72 into master May 25, 2018
@stefan-kolb stefan-kolb deleted the feature/fix-dash-formatter branch May 25, 2018 09:03
Siedlerchr added a commit that referenced this pull request May 27, 2018
…leType

* upstream/master: (33 commits)
  Import inspection uses now same font size setttings as maintable (#4062)
  Add date checker (#4007)
  Enable tests
  macOs push to application fix
  Fix #4041 to make Push to Application work again on OSX. (#4057)
  Add NormalizeEnDashesFormatter (#4045)
  Package private for tests
  Shutdown duplicate code
  Move migrations to PreferencesMigrations
  Structure startup
  Extract migrations
  A first solution for the cli problem #4038 (#4047)
  New translations JabRef_en.properties (French) (#4052)
  Update CHANGELOG.md
  Fix checkstyle
  Make test more informative in the failing case
  New translations JabRef_en.properties (French) (#4051)
  New translations JabRef_en.properties (Vietnamese) (#4050)
  Make Formatter an abstract class (and remove AbstractFormatter)
  New Crowdin translations (#4048)
  ...

# Conflicts:
#	CHANGELOG.md
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.

4 participants