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

Fix blueprints not being uploaded when shift-clicked in blueprint library #1733

Merged
merged 1 commit into from May 7, 2014
Merged

Fix blueprints not being uploaded when shift-clicked in blueprint library #1733

merged 1 commit into from May 7, 2014

Conversation

tambry
Copy link
Contributor

@tambry tambry commented May 7, 2014

Was trying to put some blueprints in blueprint library, but they well wouldn't simply appear in the list.
Came out the uploading and downloading player is only set when the slot is clicked, but not when shift clicked.
Also since the onSlotChanged is triggered always, slotClick doesn't seem to be needed anymore?

Was quite an annoying small bug.
Was trying to put some blueprints in blueprint library, but they well
wouldn't simply appear in the list.
Came out the uploading and downloading player is only set when the slot
is clicked.
@tambry tambry changed the title In library set player even when shift clicked Fix blueprints not being uploaded when shift-clicked in blueprint library May 7, 2014
@SpaceToad SpaceToad added this to the BuildCraft 6.0.11 milestone May 7, 2014
@SpaceToad SpaceToad merged commit 5781a9c into BuildCraft:6.0.x May 7, 2014
@SpaceToad
Copy link
Member

Merged, thanks for the fix! Indeed, slotClick isn't needed anymore. I removed it.

@tambry tambry deleted the librarygetnamewhenshift branch May 7, 2014 18:15
@AEnterprise
Copy link
Member

darn it, just found that bug myself to and wanted to do a pr, glad it's already fixed then

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

Successfully merging this pull request may close these issues.

None yet

3 participants