Skip to content

Change colors to match IQX White Theme#5066

Closed
abhobe wants to merge 7 commits intoQiskit:masterfrom
abhobe:5038
Closed

Change colors to match IQX White Theme#5066
abhobe wants to merge 7 commits intoQiskit:masterfrom
abhobe:5038

Conversation

@abhobe
Copy link
Contributor

@abhobe abhobe commented Sep 14, 2020

Summary

Change gate colors to match IQX Light theme.
Change Dictionaries to match font color.

Details and comments

fixes #5032

@1ucian0
Copy link
Member

1ucian0 commented Sep 15, 2020

Can you write test(s) following this style #4544 ?

@abhobe
Copy link
Contributor Author

abhobe commented Sep 15, 2020

@1ucian0 Binder
Is this what you want?

@1ucian0
Copy link
Member

1ucian0 commented Sep 17, 2020

How this relates with the propousal from #5063 (comment) ?

@abhobe
Copy link
Contributor Author

abhobe commented Sep 17, 2020

5063 solves this but does not fully match the style of IQX. The quantum and classical gates on IQX have white font color and the others have a black font. 5063 makes all fonts white instead. @1ucian0

@galeinston
Copy link
Contributor

5063 solves this but does not fully match the style of IQX. The quantum and classical gates on IQX have white font color and the others have a black font. 5063 makes all fonts white instead. @1ucian0

I disagree with this. Attached is my screenshot of what I tested.
Screen Shot 2020-09-17 at 6 38 24 AM

@Cryoris
Copy link
Collaborator

Cryoris commented Sep 17, 2020

I think these pull requests intend to do the same, looking at #5032 it seems #5063 started working on this before... maybe you can join the efforts but I think we should close this one.

@abhobe abhobe closed this Sep 17, 2020
@abhobe abhobe deleted the 5038 branch September 23, 2020 03:29
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.

Tweak colors to match new IQX colors

4 participants

Comments