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

docs: updates to language links #838

Merged
merged 1 commit into from
Sep 1, 2015
Merged

docs: updates to language links #838

merged 1 commit into from
Sep 1, 2015

Conversation

callmehiphop
Copy link
Contributor

closes #836

In order to provide a consistent user experience with the other gcloud-* sites, I've made some modifications to our language switcher directive to route the user to different sections of the other doc sites depending on what section they're currently browsing.

Previously if a user changed languages from either the splash screen or the actual docs, we would always route them to the api documentation. Per #836 this did not seem to be the desired functionality.

Preview

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 31, 2015
@stephenplusplus
Copy link
Contributor

I personally would expect that clicking a link to another site would just take me to its homepage, and not try to put me on the API docs, when I really just might be curious of the high-level "Oh, this is in Python, too?" I'm probably not flipping back and forth between the API docs as if I'm a user of both libraries.

But, @jgeewax's call.

@callmehiphop
Copy link
Contributor Author

I'm going to disagree with you here, since we're making an effort to brand the sites in the same way and switching languages is done via dropdown menu and not an external link, it feels less like the intent is to redirect the user but more to show them the same content for a different language.

If anything the part that I find weird is when viewing a specific api (e.g. storage) it does not link me to the python storage page, but rather the api landing page. However our apis aren't 1:1 so it actually makes sense for it to function this way.

@stephenplusplus
Copy link
Contributor

I agree drop downs vs an obviously-external sidebar link give different impressions. But, with this dropdown that is highly styled (not like our browser-default version switcher) and is an external link, I think the user not only expects to leave the site and go somewhere else, but I think that we should want them to go to their homepage and not an inner page of a place they haven't been before. Beyond that, I still don't think we have many multi-lingual, back-and-forth API doc switchers, so a solution for that problem seems like over-engineering.

Whatever the people want, though!

@callmehiphop
Copy link
Contributor Author

This is where our opinions are different, I don't think it's very obvious to the user that switching languages would navigate to a different site. The theming is the same across each site, which sort of gives the impression that when you switch languages, you haven't gone to different site, but instead a different page on the same site. While the dropdown is indeed highly stylized, nothing about the styling indicates that the links it contains are external, especially when considering we display an external link icon throughout the site for links that are external but not for the links within the dropdown.

If anything, maybe we should raise an issue in gcloud-common, because this is actually the current behavior in both gcloud-ruby and gcloud-python.

@stephenplusplus
Copy link
Contributor

¯_(ツ)_/¯ Code looks good. Merge time!

stephenplusplus added a commit that referenced this pull request Sep 1, 2015
@stephenplusplus stephenplusplus merged commit e77ab5f into googleapis:master Sep 1, 2015
sofisl pushed a commit that referenced this pull request Jan 17, 2023
sofisl pushed a commit that referenced this pull request Jan 24, 2023
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Benjamin E. Coe <[email protected]>
sofisl pushed a commit that referenced this pull request Jan 25, 2023
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Benjamin E. Coe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Link on landing page to Python sends you to the inner page
3 participants