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(editor): add link to gosling-lang.org to editor header #855

Merged
merged 3 commits into from
Mar 30, 2023

Conversation

etowahadams
Copy link
Contributor

Fix #836
Toward #

Change List

  • Add link to gosling-lang.org to editor header

image

Editor with a gist

image

Checklist

  • Ensure the PR works with all demos on the online editor
  • Unit tests added or updated
  • Examples added or updated
  • Documentation updated (e.g., added API functions)
  • Screenshots for visual changes (e.g., new encoding support or UI change on Editor)

Comment on lines -42 to -58
position: fixed;
right: 0;
top: 0;
color: #e18343;
display: inline-block;
vertical-align: middle;
padding-top: 8px;
padding-left: 8px;
padding-right: 8px;
margin-left: 10px;
margin-right: 8px;
cursor: pointer;
background-color: white;
}

.editor-nav-button {
display: inline-block;
vertical-align: middle;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not needed now that container has display:flex and align-items:center

Copy link
Member

@sehilyi sehilyi left a comment

Choose a reason for hiding this comment

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

Looks great! And thanks for removing unused styles!

editor/editor.tsx Outdated Show resolved Hide resolved
@etowahadams etowahadams changed the title feat: add link to gosling-lang.org to editor header feat(editor): add link to gosling-lang.org to editor header Mar 30, 2023
@etowahadams etowahadams merged commit 6964671 into master Mar 30, 2023
@etowahadams etowahadams deleted the etowahadams/project-link-header branch March 30, 2023 18:34
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.

Add "Gosling Project" link (gosling-lang.org) to the top right in the header
2 participants