Skip to content

Commit a924dcd

Browse files
author
Kirill Müller
committed
Merge branch 'master' into feature/839-facet-margin-2
2 parents ea53779 + 8bbeef3 commit a924dcd

File tree

7 files changed

+37
-15
lines changed

7 files changed

+37
-15
lines changed

.Rbuildignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,4 @@ visual_test
22
^.*\.Rproj$
33
^\.Rproj\.user$
44
^\.travis\.yml$
5+
^/\.gitattributes$

.gitattributes

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
/NEWS merge=union

.travis.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,26 @@ install:
1313

1414
script: ./travis-tool.sh run_tests
1515

16+
after_success:
17+
- ./travis-tool.sh install_github hadley/staticdocs
18+
- R -e "staticdocs::build_site()"
19+
1620
on_failure:
1721
- ./travis-tool.sh dump_logs
1822

1923
notifications:
2024
email:
2125
on_success: change
2226
on_failure: change
27+
28+
deploy:
29+
edge: true
30+
31+
provider: s3
32+
access_key_id: AKIAJDM6KGIZ6LSGAK4Q
33+
secret_access_key:
34+
secure: "r3vSpvVNcpvIcjWFyk+GhYG55iuhfmy3mwuQHQS2EqjT3Skd3mOojnZuRIqy60XXElK5/nfa3qt7KH0GCSd4Is28cqPwh+1PtQ0ZVXfeKefAk2whTPDI4P+rgOA+srUxZraLALap9QGmvlzJyXbqBCRdDmXUjtXKi2ONE9PrUk4="
35+
bucket: docs.ggplot2.org
36+
skip_cleanup: true
37+
local-dir: inst/web
38+
upload-dir: dev

NEWS

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ ggplot2 0.9.3.1.99
1010
* theme: new theme settings panel.margin.x and panel.margin.y (units) allow
1111
specifying horizontal and vertical gap between panels in facetted plots (for
1212
both grid and wrap). (Kirill Müller. Fixes #839)
13+
* Add `"none"` to documentation of `theme()` for parameter `legend.position`.
14+
(@krlmlr, #829)
15+
1316

1417
ggplot2 0.9.3.1
1518
----------------------------------------------------------------

R/theme.r

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@ print.theme <- function(x, ...) str(x)
114114
#' (\code{element_text}; inherits from \code{title}) \cr
115115
#' legend.title.align \tab alignment of legend title
116116
#' (number from 0 (left) to 1 (right)) \cr
117-
#' legend.position \tab the position of legends.
118-
#' ("left", "right", "bottom", "top", or two-element
117+
#' legend.position \tab the position of legends
118+
#' ("none", "left", "right", "bottom", "top", or two-element
119119
#' numeric vector) \cr
120120
#' legend.direction \tab layout of items in legends
121121
#' ("horizontal" or "vertical") \cr

man/theme.Rd

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -81,17 +81,17 @@ last example below.
8181
legend.title \tab title of legend (\code{element_text};
8282
inherits from \code{title}) \cr legend.title.align \tab
8383
alignment of legend title (number from 0 (left) to 1
84-
(right)) \cr legend.position \tab the position of
85-
legends. ("left", "right", "bottom", "top", or
86-
two-element numeric vector) \cr legend.direction \tab
87-
layout of items in legends ("horizontal" or "vertical")
88-
\cr legend.justification \tab anchor point for
89-
positioning legend inside plot ("center" or two-element
90-
numeric vector) \cr legend.box \tab arrangement of
91-
multiple legends ("horizontal" or "vertical") \cr
92-
legend.box.just \tab justification of each legend within
93-
the overall bounding box, when there are multiple legends
94-
("top", "bottom", "left", or "right")\cr
84+
(right)) \cr legend.position \tab the position of legends
85+
("none", "left", "right", "bottom", "top", or two-element
86+
numeric vector) \cr legend.direction \tab layout of items
87+
in legends ("horizontal" or "vertical") \cr
88+
legend.justification \tab anchor point for positioning
89+
legend inside plot ("center" or two-element numeric
90+
vector) \cr legend.box \tab arrangement of multiple
91+
legends ("horizontal" or "vertical") \cr legend.box.just
92+
\tab justification of each legend within the overall
93+
bounding box, when there are multiple legends ("top",
94+
"bottom", "left", or "right")\cr
9595

9696
panel.background \tab background of plotting area, drawn
9797
underneath plot (\code{element_rect}; inherits from

vignettes/development.Rmd

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,13 +103,14 @@ Pull requests will be evaluated against the following checklist:
103103

104104
```
105105
* [ ] Motivate the change in one paragraph, and include it in NEWS.
106-
Reference this issue and thank yourself.
106+
In parenthesis, reference your github user name and this issue:
107+
`(@hadley, #1234)`
107108
* [ ] Check pull request only includes relevant changes.
108109
* [ ] Use the [official style](http://adv-r.had.co.nz/Style.html).
109110
* [ ] Update documentation and re-run roxygen2
110111
* [ ] Add test, if bug in non-graphical function
112+
* [ ] Add visual test, if bug in graphical function
111113
* [ ] Add minimal example, if new graphical feature
112-
* [ ] Does not change existing behaviour
113114
```
114115

115116
These are explained in more detail below. This seems like a lot of work but don't worry if your pull request isn't perfect. It's a learning process and Winston and I will be on hand to help you out. A pull request is a process, and unless you've submitted a few in the past it's unlikely that your pull request will be accepted as is.

0 commit comments

Comments
 (0)