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

Restyle frontmatter dialog to match rest of pluto #2470

Merged
merged 2 commits into from
Apr 5, 2023

Conversation

tecosaur
Copy link
Contributor

@tecosaur tecosaur commented Feb 7, 2023

I recently used the frontmatter dialog, and noticed the styling didn't really fit with the rest of pluto at all. Assuming this is an oversight, I've restyled it.

Before

image

After

image

@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2023

Try this Pull Request!

Open Julia and type:

julia> import Pkg
julia> Pkg.activate(temp=true)
julia> Pkg.add(url="https://github.com/tecosaur/Pluto.jl", rev="restyle-frontmatter-dialogue")
julia> using Pluto

@fonsp
Copy link
Owner

fonsp commented Feb 7, 2023

Thanks!!

@fonsp
Copy link
Owner

fonsp commented Feb 16, 2023

I'm busy moving these weeks, I will get back to your PRs soon!

@tecosaur
Copy link
Contributor Author

Thanks Fons, it's good to know that they're just delayed, not ignored 🙂. Oh, and I should be able to tweak this one in response to your comments this weekend.

@tecosaur tecosaur force-pushed the restyle-frontmatter-dialogue branch from 10a104b to 61df0e3 Compare March 1, 2023 13:50
@tecosaur
Copy link
Contributor Author

tecosaur commented Mar 1, 2023

While I was making the button tweaks, I realised the proportions in both mine and Fons' screenshots looked a bit off to me. I think it needs an extra 0.5em of horizontal padding, and so I'll just add that:

image

@tecosaur tecosaur force-pushed the restyle-frontmatter-dialogue branch from 61df0e3 to ea314ee Compare March 1, 2023 13:56
@tecosaur
Copy link
Contributor Author

I'm busy moving these weeks, I will get back to your PRs soon!

@fonsp an update on "soon" would be nice if possible 🙂

I made the `x` buttons more visible

I made the tags visible in light mode

Bring back `outline` for input elements (this is an accessibility feature)
@fonsp fonsp merged commit 2bf4068 into fonsp:main Apr 5, 2023
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.

None yet

2 participants