Skip to content

feat: add new Java overview template#439

Closed
alicejli wants to merge 5 commits intogoogleapis:mainfrom
alicejli:updateJavaOverview
Closed

feat: add new Java overview template#439
alicejli wants to merge 5 commits intogoogleapis:mainfrom
alicejli:updateJavaOverview

Conversation

@alicejli
Copy link
Copy Markdown
Contributor

Fix for #436

parent bug: b/278903399
Corresponds to [R9] Confusing ‘On This Page’ Indentation on go/cloud-java-rad-refactor and slide 20 on go/cloud-java-rad-visual-review.

For example on: https://cloud.google.com/java/docs/reference/google-cloud-apikeys/latest/overview

APIKeysClient is outdented even though it is a child of com.google.api.apikeys.v2.

This will be a two-part fix, with this being the template change, and googleapis/java-docfx-doclet#178 being the doclet change

Comment thread testdata/goldens/java/overview.html Outdated
Comment on lines +13 to +15
<h2 id="packages">
</h2>
<h2><a class="xref" href="com.google.cloud.speech.v1.html">com.google.cloud.speech.v1</a></h2>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

com.google.cloud.speech.v1 is a package, so do we know what the above and empty <h2 id="packages"> is meant to be used for?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah. Understood. Thank you.

@alicejli
Copy link
Copy Markdown
Contributor Author

Messed up rebasing this PR - will open a new one.

@alicejli alicejli closed this May 24, 2023
@alicejli alicejli deleted the updateJavaOverview branch May 24, 2023 14:52
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.

5 participants