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

Fix broken indents by nested list bullets #1457

Conversation

grauwoelfchen
Copy link
Contributor

I've just found a broken indents arround a section relates to access block in bigquery_dataset.html.markdown file πŸ™‹β€β™€οΈ

Problem

Current website shows the section like below. This may be a bit confusing:

current-website-has-bit-weird-indents-20191203

Although it looks fine on GitHub...

On GitHub

on-github-markdown-seems-fine-20191203

Fix

Then, I've just removed a line break before a paragraph by 18160d5.
# I've checked the markdown rendering using redcarpet on local.

Before
Rendering result

rendering-result-before-this-change-20191203

After
Rendering result

rendering-result-by-this-change-20191203

I hope that it works.

@ghost ghost added the size/xs label Dec 3, 2019
@ghost ghost requested a review from danawillow December 3, 2019 10:34
@ghost ghost added the documentation label Dec 3, 2019
@ghost ghost requested a review from tysen December 3, 2019 10:34
Copy link
Contributor

@danawillow danawillow 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 catching this @grauwoelfchen! This file is actually autogenerated from https://github.com/GoogleCloudPlatform/magic-modules, so I'll prepare a PR there that'll make the same change. Feel free to make any future changes there as well :)

@grauwoelfchen
Copy link
Contributor Author

@danawillow Oh, it's autogenerated! πŸ˜… ... and you've merged also my useless change into above PR πŸ™‡β€β™€οΈ πŸ’¦
Thank you so much for your work ;)

@danawillow
Copy link
Contributor

Eh I wanted to at least give you some credit :)

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