Skip to content
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

Added image for auto-follow-topics #422

Merged
merged 3 commits into from
Apr 9, 2024

Conversation

Waakul
Copy link
Contributor

@Waakul Waakul commented Mar 7, 2024

addon id: auto-follow-topics
addon name Follow topics after posting

@Samq64
Copy link
Member

Samq64 commented Mar 7, 2024

Addon image edits should be kept to a minimum. The second button and arrow are fine but the circle and text seem unnecessary. The image should just be zommed in on the buttons instead.

@Waakul
Copy link
Contributor Author

Waakul commented Mar 8, 2024

Addon image edits should be kept to a minimum. The second button and arrow are fine but the circle and text seem unnecessary. The image should just be zommed in on the buttons instead.

if it's zoomed on the button then it's meaningless as post = follow, so the photo of posting is neccessary.

@Waakul
Copy link
Contributor Author

Waakul commented Mar 8, 2024

auto-follow-topics (2)
should i edit the details and content out?

@Samq64
Copy link
Member

Samq64 commented Mar 8, 2024

if it's zoomed on the button then it's meaningless

Right. Maybe it could show the mouse cursor hovering over the post button?

@Waakul
Copy link
Contributor Author

Waakul commented Mar 8, 2024

if it's zoomed on the button then it's meaningless

Right. Maybe it could show the mouse cursor hovering over the post button?

Then it wont follow the post yet?

@Waakul
Copy link
Contributor Author

Waakul commented Mar 8, 2024

if it's zoomed on the button then it's meaningless

Right. Maybe it could show the mouse cursor hovering over the post button?

I have a simillar idea, but that'd require alot of editing

@Waakul
Copy link
Contributor Author

Waakul commented Mar 8, 2024

auto-follow-topics (3)
Is this ok?

@Waakul
Copy link
Contributor Author

Waakul commented Mar 8, 2024

Less edited version of the above
auto-follow-topics (4)

@Waakul
Copy link
Contributor Author

Waakul commented Mar 8, 2024

auto-follow-topics (5)
I guess i'll stick with this one

@Samq64
Copy link
Member

Samq64 commented Mar 8, 2024

auto-follow-topics (5)
I guess i'll stick with this one

When I said zoomed in I didn't mean that much but that's fine.

@Waakul
Copy link
Contributor Author

Waakul commented Apr 7, 2024

Is this gonna merge soon?

Copy link
Member

@Hans5958 Hans5958 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really nice. I would say to keep the right arrow on a circle like some of the previous images, but this would do for now.

Also, I saw that your early version and I would like to remind you which where are the image shown, for example, the Addons page. It wouldn't look nice if one needs to zooms that in by opening the image.

@Hans5958 Hans5958 merged commit 365cf72 into ScratchAddons:master Apr 9, 2024
@Hans5958
Copy link
Member

Hans5958 commented Apr 11, 2024

Related: #4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants