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 R icon size #59891

Merged
merged 1 commit into from Oct 4, 2018

Conversation

Projects
None yet
3 participants
@yeahok
Contributor

yeahok commented Oct 3, 2018

Fixes #57595

@RMacfarlane RMacfarlane assigned misolori and aeschli and unassigned misolori Oct 3, 2018

@misolori

This comment has been minimized.

Member

misolori commented Oct 3, 2018

Wouldn't this need to be fixed upstream in order for us to update our version? I see there's a PR for this but it's not yet merged.

@yeahok

This comment has been minimized.

Contributor

yeahok commented Oct 3, 2018

Yeah I made the other pull request too with the same fix. I probably jumped the gun a bit. Should this pull request be closed?

@aeschli

This comment has been minimized.

Contributor

aeschli commented Oct 4, 2018

@yeahok Cool, thanks a lot. I've seen you filed jesseweed/seti-ui#509.
As jesseweed/seti-ui uually takes a while to go through the PRs, I already push it here.

@aeschli aeschli merged commit a7b8fdb into Microsoft:master Oct 4, 2018

1 check passed

license/cla All CLA requirements met.

@aeschli aeschli added this to the October 2018 milestone Oct 4, 2018

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