Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1108)

Issue 8509043: Remove remaining uses of \times (3301) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years ago by Trevor Daniels
Modified:
11 years ago
Reviewers:
Graham Percival, janek, dak
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Remove remaining uses of \times (3301) In English language docs only (thanks to Federico) Also use @dots{} instead of ... This is in a separate commit.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Respond to David's comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -26 lines) Patch
M Documentation/learning/fundamental.itely View 1 10 chunks +26 lines, -26 lines 0 comments Download

Messages

Total messages: 5
dak
https://codereview.appspot.com/8509043/diff/1/Documentation/learning/fundamental.itely File Documentation/learning/fundamental.itely (right): https://codereview.appspot.com/8509043/diff/1/Documentation/learning/fundamental.itely#newcode502 Documentation/learning/fundamental.itely:502: between or across notes: ties (marked by a tilde, ...
11 years ago (2013-04-08 09:59:06 UTC) #1
Trevor Daniels
Respond to David's comment
11 years ago (2013-04-08 11:42:57 UTC) #2
Graham Percival
LGTM
11 years ago (2013-04-09 07:23:21 UTC) #3
janek
LGTM
11 years ago (2013-04-13 14:03:54 UTC) #4
Trevor Daniels
11 years ago (2013-04-14 21:48:15 UTC) #5
Pushed to staging as
bb994e1c15cd048c12ab3d777e90b447bbf37b16
but with a commit message 
(Doc: use @dots{} rather than "...") 
that does not match the original issue.
Closing ...
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b