Skip to content

Conversation

@imklau
Copy link
Contributor

@imklau imklau commented Mar 3, 2020

Related issues (optional)

Closes: #519

@imklau imklau requested a review from sanscontext March 3, 2020 12:52
@imklau imklau self-assigned this Mar 3, 2020
@imklau
Copy link
Contributor Author

imklau commented Mar 3, 2020

Just to be sure,
the hide-feedback field should be responsible for hiding both: feedback buttons and Github links?

and what about the copy in the right nav: should be "Was this page helpful?" or "Is this page helpful?"
I think it doesn't look really great with "Was" because there's too much text that doesn't fit in the one line.

@sanscontext

@sanscontext
Copy link
Contributor

@noeemi Hm, I think for now we can make hide-feedback turn off both, since that'll mostly be used in landing pages now that the Legal docs are out of the repo. (I can probably figure out how to separate them later if needed.)

For the text: "Was this helpful" is more grammatically correct in English, but I agree that it's weird that it spills over into a new line. Is there a different way to solve this without changing "was"?

@imklau
Copy link
Contributor Author

imklau commented Mar 4, 2020

Hm, I think for now we can make hide-feedback turn off both, since that'll mostly be used in landing pages now that the Legal docs are out of the repo. (I can probably figure out how to separate them later if needed.)

okay, sounds good to me!

For the text: "Was this helpful" is more grammatically correct in English, but I agree that it's weird that it spills over into a new line. Is there a different way to solve this without changing "was"?

The design contains "Is this page helpful?" too and I thought maybe I made the mistake but now I see :)

Our sidebar:
Zrzut ekranu 2020-03-4 o 10 00 43
has 144px width so the best way to solve this would be to increase the size to 150px. It's just 6px more but in this case, it's enough. (and it's a really quick fix :D)
Zrzut ekranu 2020-03-4 o 10 03 35

@netlify
Copy link

netlify bot commented Nov 19, 2020

Deploy preview for segment-docs-preview ready!

Built with commit a524c51

https://deploy-preview-676--segment-docs-preview.netlify.app

@markzegarelli
Copy link
Contributor

closing. This was brought up to date in #1584

@markzegarelli markzegarelli deleted the 519-github-side-links branch August 4, 2021 18:10
bot-docsteam added a commit that referenced this pull request Nov 10, 2022
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.

Clean up/restyle "edit in github"/"request changes" links

4 participants