Skip to content

Update README.md with Fortran-Lang Discourse #504

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

Merged
merged 6 commits into from
Jun 1, 2022
Merged

Conversation

gnikit
Copy link
Member

@gnikit gnikit commented Jun 1, 2022

Fixes #485

Added GitHub Discussions & Discourse link

TODO:

  • Add CSS in SVG for hover Not possible with GitHub Markdown

@awvwgk
Copy link
Member

awvwgk commented Jun 1, 2022

General tip for buttons or labels, try to keep sufficient space from the border. The usual distance measure should be at least the size of a letter o.

@awvwgk
Copy link
Member

awvwgk commented Jun 1, 2022

Regarding colors, there are more than just the Fortran purple, a palette I've been using for Fortran related projects can be found at fortran-lang/assets#3.

@gnikit
Copy link
Member Author

gnikit commented Jun 1, 2022

Regarding colors, there are more than just the Fortran purple, a palette I've been using for Fortran related projects can be found at fortran-lang/assets#3 (comment).

Thanks @awvwgk for the tip, although the spacings should be alright. What they are not is properly centered, but I am not sure if that is a bad thing since the README is generally left justified.

I will have a look for the color pallete, might change a few "buttons" around. Also it would be a good idea to design a logo for FPM and stdlib

@gnikit gnikit linked an issue Jun 1, 2022 that may be closed by this pull request
@awvwgk
Copy link
Member

awvwgk commented Jun 1, 2022

Also it would be a good idea to design a logo for FPM and stdlib

We have an fpm logo, checkout fortran-lang/fpm#380 (comment). I just owe the author to add it to the fpm documentation.

@gnikit gnikit force-pushed the gnikit/issue485 branch from 6937fab to b094d25 Compare June 1, 2022 14:47
@gnikit
Copy link
Member Author

gnikit commented Jun 1, 2022

Also it would be a good idea to design a logo for FPM and stdlib

We have an fpm logo, checkout fortran-lang/fpm#380 (comment). I just owe the author to add it to the fpm documentation.

Awesome I will update it now.

@gnikit
Copy link
Member Author

gnikit commented Jun 1, 2022

Do you think it would be a good idea to start a repo with all the artwork assets? I find myself looking for the SVGs quite often, so it would be more convenient if we could use a GitHub raw URL instead.

Fixes #485

Added GitHub Discussions & Discourse link
@gnikit gnikit force-pushed the gnikit/issue485 branch from b094d25 to d40e693 Compare June 1, 2022 17:20
@gnikit gnikit force-pushed the gnikit/issue485 branch from d40e693 to 34fa2c6 Compare June 1, 2022 17:32
@awvwgk
Copy link
Member

awvwgk commented Jun 1, 2022

image

Best convert the text to a path, otherwise the svg will look strange if the correct fonts are not available.

@gnikit
Copy link
Member Author

gnikit commented Jun 1, 2022

image

Best convert the text to a path, otherwise the svg will look strange if the correct fonts are not available.

Completely forgot about that (and that is why I had a pngs before), nice catch. This is basically done (after converting to paths). Do you think we need to add anything more?

PS I accidentally edited your comment apologies for that

@gnikit gnikit merged commit e69709c into master Jun 1, 2022
@gnikit gnikit deleted the gnikit/issue485 branch June 1, 2022 20:54
@gnikit gnikit added this to the v3.0 milestone Jun 1, 2022
@gnikit
Copy link
Member Author

gnikit commented Jun 3, 2022

I just realised that VSCE i.e. VS Code Marketplace, has banned SVGs since they can be a security vulnerability so I will have to convert all the SVGs to high quality PNGs anyway.

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.

Update README Update README.md with Fortran-Lang Discourse
2 participants