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

Update links to source code, fix heading levels #365

Merged
merged 6 commits into from
Nov 20, 2024

Conversation

zbynek
Copy link
Contributor

@zbynek zbynek commented Aug 3, 2024

The "edit on GitHub" link is not positioned correctly on pages that use <h2> instead of <h1> and it points to a wrong branch.

While fixing that, I also updated some links that point to googlesource and should point to GH.

@@ -43,30 +43,30 @@ You build and run LogExample the same way you would build and run any of the oth

[![Logging Example web page](images/LoggingExample.png)](images/LoggingExample.png)

LogExample is configured using [`LogExample.gwt.xml`](https://gwt.googlesource.com/gwt/+/master/samples/logexample/src/com/google/gwt/sample/logexample/LogExample.gwt.xml).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Links are dead since gwtproject/gwt@54908f2

Copy link
Member

Choose a reason for hiding this comment

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

Probably should phase this out entirely, update the doc with what is expected instead of linking to a defunct example (in a later commit)

@@ -61,7 +61,7 @@ Google App Engine Datastore. In persistence frameworks like JDO and JPA,
entities are annotated with @Entity. RequestFactory does not require the use of
any particular framework or annotations on your domain classes. Here's part of
an entity definition from the [Expenses
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zbynek
Copy link
Contributor Author

zbynek commented Aug 30, 2024

@@ -43,30 +43,30 @@ You build and run LogExample the same way you would build and run any of the oth

[![Logging Example web page](images/LoggingExample.png)](images/LoggingExample.png)

LogExample is configured using [`LogExample.gwt.xml`](https://gwt.googlesource.com/gwt/+/master/samples/logexample/src/com/google/gwt/sample/logexample/LogExample.gwt.xml).
Copy link
Member

Choose a reason for hiding this comment

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

Probably should phase this out entirely, update the doc with what is expected instead of linking to a defunct example (in a later commit)

@niloc132 niloc132 merged commit 4746df9 into gwtproject:main Nov 20, 2024
@niloc132
Copy link
Member

niloc132 commented Nov 20, 2024

(commented on the wrong PR)

@zbynek zbynek deleted the links branch November 21, 2024 00:05
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.

2 participants