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

Remove secondary icon for brushes #53

Closed
Viza74 opened this Issue Jul 30, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@Viza74

Viza74 commented Jul 30, 2018

I regret to inform you about this very serious bug.
All the other tools in the left toolbar have a tooltip except the brush tool. It is pretty sad.

Is there a more serious category for bugs than a "crash" bug?
Because this issue is definitely that.

;)

@nurpax

This comment has been minimized.

Show comment
Hide comment
@nurpax

nurpax Jul 30, 2018

Owner

:) I dropped it because it messed up with the sub-tool selector.

But @Esshahn opined that maybe the sub-tool menu is sort of redundant anyway as the brush selection can be cleared with ESC. Or maybe first click on the brush menu would enable brush paint mode if there's a brush selected and another click would clear the brush and go into brush selection mode.

Owner

nurpax commented Jul 30, 2018

:) I dropped it because it messed up with the sub-tool selector.

But @Esshahn opined that maybe the sub-tool menu is sort of redundant anyway as the brush selection can be cleared with ESC. Or maybe first click on the brush menu would enable brush paint mode if there's a brush selected and another click would clear the brush and go into brush selection mode.

@nurpax nurpax added this to the 0.2 milestone Jul 30, 2018

@Esshahn Esshahn changed the title from No tooltip for the brush tool to Remove secondary icon for brushes Aug 4, 2018

@Esshahn

This comment has been minimized.

Show comment
Hide comment
@Esshahn

Esshahn Aug 4, 2018

Collaborator

(issue renamed for clarity)

  • remove the secondary (mousedown) icon from the tools menu as ESC handles that now
  • add tooltip to primary icon

screenshot 2018-08-04 um 10 28 24

Collaborator

Esshahn commented Aug 4, 2018

(issue renamed for clarity)

  • remove the secondary (mousedown) icon from the tools menu as ESC handles that now
  • add tooltip to primary icon

screenshot 2018-08-04 um 10 28 24

@Esshahn Esshahn modified the milestones: 0.2, 0.3 Aug 4, 2018

@nurpax nurpax closed this in 85be06d Aug 4, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment