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

Make the “raw data” button optional #92

Merged
merged 3 commits into from
Jan 23, 2018

Conversation

pluehne
Copy link
Contributor

@pluehne pluehne commented Jan 23, 2018

This introduces a new configuration option “showRawDataLink,” through which the “raw data” button can be explicitly hidden (requested in #87).

This is used for hiding the “raw data” download button in the collaboration chart, where the raw data isn’t useful to view or edit in a spreadsheet application.

Also, the margins of chart titles were incorrect in the case that the corresponding action bar was hidden. This change applies the same style to all titles, regardless of whether an action bar is shown or not.

See how the “raw data” button is missing in the screenshot below 😄:

screenshot from 2018-01-23 14-19-05

@codecov-io
Copy link

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is 0%.

@larsxschneider
Copy link
Collaborator

Looks good! Minor nit: can you squash de83dbe and 118c075 ?

@pluehne
Copy link
Contributor Author

pluehne commented Jan 23, 2018

@larsxschneider: Good idea, the commits belong together. I’ll squash them!

This introduces a new configuration option “showRawDataLink,” through
which the “raw data” button can be explicitly hidden.

This is used for hiding the “raw data” download button in the
collaboration chart, where the raw data isn’t useful to view or edit in
a spreadsheet application.

Additionally, the documentation is extended to cover the new option.
The margins of chart titles were incorrect in the case that the
corresponding action bar was hidden. This change applies the same style
to all titles, regardless of whether an action bar is shown or not.
@pluehne pluehne force-pushed the patrick/optional-raw-button-link branch from 118c075 to 864ade5 Compare January 23, 2018 14:58
@larsxschneider larsxschneider merged commit d63504a into master Jan 23, 2018
@larsxschneider larsxschneider deleted the patrick/optional-raw-button-link branch January 23, 2018 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants