activist icon indicating copy to clipboard operation
activist copied to clipboard

Rework UI for first organization, group and event main image change

Open andrewtavis opened this issue 7 months ago • 11 comments

Terms

Description

As of now left sidebar images for organizations, groups and events have an icon and a model to change their image (as of now nonfunctional). It was decided yesterday that this behavior is confusing to the user. It would be best if this + button on the left sidebar icons that triggers the image model would instead be removed and replaced with an optional CTA button that appears below the left sidebar icon whenever there isn't an icon added by a user. When there is an icon added by a user then it would need to be removed in the settings once the work in #1200 is finished :)

Contribution

Happy to support with this, pick it up or review as needed!

andrewtavis avatar Mar 31 '25 21:03 andrewtavis

Hi, if help is still needed for this issue, I would like to contribute to this issue.

celine36 avatar Apr 02 '25 05:04 celine36

Sure thing, @celine36! I'll assign you now :) Please let us know if you need any assistance! 😊

andrewtavis avatar Apr 02 '25 11:04 andrewtavis

Hi @andrewtavis, thanks

celine36 avatar Apr 03 '25 07:04 celine36

Hi @andrewtavis, do I make a fork to work on the code?

celine36 avatar Apr 04 '25 07:04 celine36

Yes exactly, @celine36 :)

andrewtavis avatar Apr 04 '25 11:04 andrewtavis

The contributing guide should have the needed info 😊

andrewtavis avatar Apr 04 '25 11:04 andrewtavis

Hi @andrewtavis, thanks

celine36 avatar Apr 07 '25 02:04 celine36

Hi @andrewtavis, I tried a couple of small changes on a button tag but could not see it when I used Docker. Do I have to git commit the changes when using Docker to see if they worked?

celine36 avatar Apr 17 '25 06:04 celine36

Hey @celine36 👋 The changes should be reflected and hot reloaded into the frontend when you have Docker up and running with docker compose --env-file .env.dev up --build :) Maybe try changing something basic like removing a component from a page and see if it's reflected on the page? Try reloading the page as hot reload could be malfunctioning 🤔 Please let me know if you're continuing to experience problems here and we'll support!

andrewtavis avatar Apr 17 '25 10:04 andrewtavis

Hi @andrewtavis, thanks, it works now, I didn't make the change that I thought I did

celine36 avatar Apr 18 '25 10:04 celine36

Great to hear, @celine36! Please let us know if you need more support :)

andrewtavis avatar Apr 18 '25 10:04 andrewtavis

Hi, @andrewtavis, I have made the layout changes to the button for the image upload for each event and each organisation in the left sidebar, to push them back to celine36/activist, do I use git commit and then git push?

celine36 avatar May 03 '25 10:05 celine36

Yes you can git commit and git push and then you should be able to open a pull request on the repository here 😊 Looking forward to the review!

andrewtavis avatar May 03 '25 13:05 andrewtavis

Closed by #1248 :) Thanks for the contribution here, @celine36!

@mattburnett-repo, for now we have a CTA button under the images, which is always present as we're not loading the entities into the sidebar index to then check if the image is there (as you know).

andrewtavis avatar May 08 '25 14:05 andrewtavis