-
Notifications
You must be signed in to change notification settings - Fork 345
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
Improve the Community Page #1270
Comments
@patricoferris Thank you so much for assigning me this task. I would definitely give it a try. |
Hey @patricoferris, I have worked on 2.) and made the icons symmetrical and of same size. End result looks something like this. I have also created a PR for the same and it has passed deployability checks. |
Hey @patricoferris I am unclear about this part, Do you want me to replace meetings with OCaml workshops? If yes, can you please let me know, from where shall I get the list of latest 3 OCaml workshops? |
Hi @Kanak1081 -- great work so far :)) Yes I should have been clearer - the full list of meetings can be found on https://ocaml.org/meetings/ (in the code it is https://github.com/ocaml/ocaml.org/blob/master/site/meetings/index.md). I just meant using the latest three events from that list instead of ones from a longer time ago. Hopefully that clears things up? |
Thanks @patricoferris :) for your kind help. I have made certain commits for updating the community page on the above mentioned issues by you. I have updated the OCaml meetings in Community page with the latest three and styled them. Please have a look at the end results in different screen sizes. I would be happy to know your suggestions. Thanks :) |
Hey @patricoferris can I take up this issue? |
Hi @DivyanshiSingh, @Kanak1081 is currently working on this and there is a PR open for it ocaml/ocaml.org#1272. |
Hi @Kanak1081, I saw that you closed your PR, do you still want to work on this? Thanks! :) |
There are a few little issues with the community page that I think need tweaking.
fr
version also includes dates whilst theen
version doesn't. I think the dates look good.OCaml around the Web
section could be a little neater (e.g. same size images). I'm also open to any layout improvements too.@Kanak1081 would you be interested in this issue as part of #1245, let me know either way below please :))
The text was updated successfully, but these errors were encountered: