Skip to content

Fix link - #15567

Merged
aevesdocker merged 3 commits into
docker:masterfrom
aevesdocker:fix-link
Sep 5, 2022
Merged

Fix link#15567
aevesdocker merged 3 commits into
docker:masterfrom
aevesdocker:fix-link

Conversation

@aevesdocker

Copy link
Copy Markdown
Contributor

No description provided.

@aevesdocker aevesdocker mentioned this pull request Sep 5, 2022
@crazy-max

Copy link
Copy Markdown
Member

Yeah maybe hardcoded link is better atm

@aevesdocker
aevesdocker enabled auto-merge (squash) September 5, 2022 14:26
@crazy-max

crazy-max commented Sep 5, 2022

Copy link
Copy Markdown
Member

I will take a look to have a "better" degraded mode for GitHub users but LGTM in the meantime.

@netlify

netlify Bot commented Sep 5, 2022

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit a643a81
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/631605ce573c1f000835abe0
😎 Deploy Preview https://deploy-preview-15567--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@aevesdocker
aevesdocker merged commit 9b95b4d into docker:master Sep 5, 2022
@crazy-max

crazy-max commented Sep 5, 2022

Copy link
Copy Markdown
Member

Also a bit out of scope with this PR but in dark mode, icons don't look great: https://docs.docker.com/contribute/overview/

image

We should have light and dark mode icons I think.

Edit: Looks like it's svg so I guess component-icon css class should set the proper color pattern instead.

@aevesdocker

aevesdocker commented Sep 5, 2022

Copy link
Copy Markdown
Contributor Author

@crazy-max yes, aware the icons need changing. Am working with one of the designers to get this sorted asap and align with the Design system

@crazy-max

Copy link
Copy Markdown
Member

Awesome thx @aevesdocker

@aevesdocker
aevesdocker deleted the fix-link branch September 12, 2022 15:51
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.

3 participants