Install/Remove button in Mode/Library/Tool managers not working in the current source #3172

Closed
REAS opened this Issue Apr 3, 2015 · 6 comments

Comments

Projects
None yet
4 participants
@REAS
Member

REAS commented Apr 3, 2015

OS X 10.8.5, running from source.

The install button is there and it highlights on rollover, but there is no action when it's pressed.

@REAS REAS changed the title from Mode Manager not working in 0233 to Mode and Mode Managers not working in 0233 Apr 3, 2015

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Apr 4, 2015

Member

@joelmoniz any thoughts on what's happening here?

Member

benfry commented Apr 4, 2015

@joelmoniz any thoughts on what's happening here?

@benfry benfry added the help wanted label Apr 4, 2015

@Plastix

This comment has been minimized.

Show comment
Hide comment
@Plastix

Plastix Apr 4, 2015

Contributor

I am replicating this on OSX 10.10 and running latest commit from master. Will investigate.

Contributor

Plastix commented Apr 4, 2015

I am replicating this on OSX 10.10 and running latest commit from master. Will investigate.

@Plastix

This comment has been minimized.

Show comment
Hide comment
@Plastix

Plastix Apr 4, 2015

Contributor

I figured out that you have to click the install/remove button (so it is selected) and press space to actually send the click event. No idea why...

Contributor

Plastix commented Apr 4, 2015

I figured out that you have to click the install/remove button (so it is selected) and press space to actually send the click event. No idea why...

@joelmoniz

This comment has been minimized.

Show comment
Hide comment
@joelmoniz

joelmoniz Apr 4, 2015

Member

@benfry
Terribly sorry. I'm afraid I've not yet been able to look into this, since I've had loads of work at school his last week. Will definitely be able to look into this tomorrow.

On a side note, based on @Plastix 's tests, might this be related to #3079? @Plastix could you please let me know if double clicking the install/remove button enables installation/removal?

Thanks!

Member

joelmoniz commented Apr 4, 2015

@benfry
Terribly sorry. I'm afraid I've not yet been able to look into this, since I've had loads of work at school his last week. Will definitely be able to look into this tomorrow.

On a side note, based on @Plastix 's tests, might this be related to #3079? @Plastix could you please let me know if double clicking the install/remove button enables installation/removal?

Thanks!

@Plastix

This comment has been minimized.

Show comment
Hide comment
@Plastix

Plastix Apr 4, 2015

Contributor

@joelmoniz No matter how many times I click the buttons it doesn't install/remove the library. The only way I've found is to click the button (so it is selected) and press space. Code is all over the place in ContributionPanel.java and it is hard to tell what is going wrong.

Here is a gif:
test

Contributor

Plastix commented Apr 4, 2015

@joelmoniz No matter how many times I click the buttons it doesn't install/remove the library. The only way I've found is to click the button (so it is selected) and press space. Code is all over the place in ContributionPanel.java and it is hard to tell what is going wrong.

Here is a gif:
test

@benfry benfry changed the title from Mode and Mode Managers not working in 0233 to Install/Remove in Mode/Library/Tool managers not working in 0233 Apr 4, 2015

@joelmoniz

This comment has been minimized.

Show comment
Hide comment
@joelmoniz

joelmoniz Apr 4, 2015

Member

Oh wow. Thanks a bunch. The behavior on Windows isn't like this at all, although I'm fairly certain that the same bug in the Contributions Manager is at the root of both these issues. I shall definitely have a look at this asap.

Thanks again.

Member

joelmoniz commented Apr 4, 2015

Oh wow. Thanks a bunch. The behavior on Windows isn't like this at all, although I'm fairly certain that the same bug in the Contributions Manager is at the root of both these issues. I shall definitely have a look at this asap.

Thanks again.

@benfry benfry changed the title from Install/Remove in Mode/Library/Tool managers not working in 0233 to Install/Remove button in Mode/Library/Tool managers not working in the current source Apr 4, 2015

@benfry benfry added the critical label Apr 26, 2015

@benfry benfry closed this in 4db151c Apr 27, 2015

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