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

feat: figures with open in tab link #1030

Merged
merged 2 commits into from
Aug 3, 2020
Merged

feat: figures with open in tab link #1030

merged 2 commits into from
Aug 3, 2020

Conversation

olizilla
Copy link
Collaborator

@olizilla olizilla commented Jul 28, 2020

In the main branch you can open diagrams in a new tab. This PR brings them back to beta in a nicer form:

  • Improve styling of embedded svgs.
  • Use semantic figure and figurecaption elements.
  • Add open in tab link
  • Pull svgs and source files into the intro section as appropriate

Screenshot 2020-07-29 at 13 00 27

License: MIT
Signed-off-by: Oli Evans oli@tableflip.io

In the main branch you can open diagrams in a new tab. This commit explores bringing them back.

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
@olizilla
Copy link
Collaborator Author

@hugomrdias i know you've thought about this... did you have a solution in mind?

@hugomrdias
Copy link
Contributor

im not seeing the question, sorry can you explain further please

@olizilla
Copy link
Collaborator Author

@hugomrdias There used to be links next to diagrams that let the user open them in a new tab, which seems like a good thing... You mentioned this in passing in a conversation, so I wondered if you had a plan for bringing back the "open diagram in new tab" links. This PR is just a proposal.

- Improve styling of embedded svgs.
- Use semantic figure and figurecaption elements.
- Add open in tab link
- Pull svgs and source files into the intro section as appropriate

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
@olizilla olizilla changed the title wip: brink back open diagram in tab link feat: figures with open in tab link Jul 29, 2020
@olizilla olizilla marked this pull request as ready for review July 29, 2020 12:06
@olizilla olizilla requested a review from hugomrdias July 29, 2020 12:06
@olizilla olizilla added this to In progress in beta.spec.filecoin.io via automation Jul 29, 2020
@olizilla olizilla added the hint: beta Hint: Issues related to the beta branch label Jul 29, 2020
@olizilla olizilla merged commit 49b2149 into beta Aug 3, 2020
beta.spec.filecoin.io automation moved this from In progress to Done Aug 3, 2020
@olizilla olizilla deleted the open-diagram-link branch August 3, 2020 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hint: beta Hint: Issues related to the beta branch
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants