search

publiclab / infragram

A minimal core of the Infragram.org project in JavaScript
https://infragram.org/sandbox/
GNU General Public License v2.0
46 stars 165 forks source link

Make toolbar use flex #311

Open erayalkis opened 2 years ago

erayalkis commented 2 years ago

This issue makes the toolbar use flex to center the elements to improve user experience and make it easier to maintain, since flex makes it super easy to mess around with the design.

welcome[bot] commented 2 years ago

Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help. Dangerbot will test out your code and reply in a bit with some pointers and requests. There may be some errors, but don't worry! We'll work through them with you! 👍🎉😄
One thing that can help to get started is to make sure you've included a link back to the original issue you're solving, in the format fixes #0000 (for example). And to make sure the PR title describes what you're trying to do! (often it can be the same as the issue title) Thanks! 🙌
Then, you can say hello in our chatroom & share a link to this PR to get a review! 👋 ✅

gitpod-io[bot] commented 2 years ago

cesswairimu commented 2 years ago

Hi @erayalkis, thanks for working on this. Was there an issue for this? If so, kindly link us to it. Also please add some screenshots of the before and after to see the changes. Thanks

erayalkis commented 2 years ago

My bad, I knew I was forgetting something !! There wasn't any specific issue to link, I saw that people were talking about redesign ideas and I thought that switching to flex for the toolbar would make things easier to update and maintain. I'll attach the screenshots now, let me know if I need to do anything else!

image image