pyOpenSci / pyopensci.github.io

This is the website for pyOpenSci. Contributions welcome!!
https://www.pyopensci.org
Other
36 stars 61 forks source link

Fix: partner page - align styles across pages #295

Closed lwasser closed 8 months ago

lwasser commented 8 months ago

This PR cleans up the partner page - specifically it ensures less inline html and aligns styles between this page and the peer review page which we are working on #287

@kierisi i'm going to ask you to review this one as well!! any and all feedback welcome here. we can totally redo the text as needed i'm not wedded to it.

Adding a TODO list from our canva board


lwasser commented 8 months ago

@kierisi this pr is a bit trickier in that i didn't edit the entire file - the gif below should show you how you can view additional lines of text in the file. then just leave comments for things you think should be changed lower down in the pull request if you can't edit it inline and suggest changes as you did in the peer review page.

feel free to ask if this approach isn't clear - but i'd love your review of the entire page!! thank you. github-text

lwasser commented 8 months ago

here is a rendered version of this page again all feedback is welcome.

lwasser commented 8 months ago

i am done tweaking - i just simplified both of the partner page graphics. @willingc thank you for the alt tag note - we have to add those :) i just wanted to make sure people liked the diagrams first!!! but i'm stopping work on the diagrams now for the evening. i'm hoping what i just pushed makes for a much cleaner / readable graphic. they felt too complex to me for the small space they were in.

lwasser commented 8 months ago

most recent version of the page rendered is here i'm done pushing to this pr for the evening! promise :)

kierisi commented 8 months ago

@lwasser I can't seem to add suggestions to most of the file, as it's greyed out. any suggestions?

Screenshot 2023-12-05 at 9 25 57 AM
lwasser commented 8 months ago

@lwasser I can't seem to add suggestions to most of the file, as it's greyed out. any suggestions? Screenshot 2023-12-05 at 9 25 57 AM

@kierisi yes - that is because i didn't edit the top of the page. i have a few ideas.

  1. you could do it via a pr to my fork but
  2. it might be easier if we move to a hackmd file if you want to edit suggest things throughout? i wouldn't copy the images there but just the markdown for easy editing
  3. the last option is you add comments that suggest changes to the file here individually rather than inline. I can integrate them locally.

i'm open to whatever way you're prefer to move forward here!! i know this isn't ideal. you can only add inline comments on changes i've made to specific lines but i know the entire page needs work.

lwasser commented 8 months ago
Screenshot 2023-12-08 at 5 17 23 PM

@kierisi i forgot that we can add buttons to the template sections on the website !! we could even add font awesome icons in the future if we wish to!!! but i've gone through and added more "call to actions" in all of the sections following our web flow conversations!! and now i'll stop using so many !!!!!!! :)

lwasser commented 8 months ago

ok i'm really happy with this page now. @kierisi have a look when you have a moment at the final rendered version and let me know what you think!! i think we can merge this next week and call this task complete!

kierisi commented 8 months ago

Looks good to me! I say 🚢 when you're ready!

lwasser commented 8 months ago

ok cool. and i didn't full understand your comment @kierisi about who we're talking to until i read it through again. you are right. maybe we can think about this as a "search and destroy" mission once we decide what "voice" second person vs third we use in our website text. Yay ok i'll go ahead and merge this... 🚢 :shipit: