Skip to content

fixed Visually out of place drop down in pull requests views with Safari#3244

Closed
Yash621 wants to merge 6 commits into
microsoft:mainfrom
Yash621:main
Closed

fixed Visually out of place drop down in pull requests views with Safari#3244
Yash621 wants to merge 6 commits into
microsoft:mainfrom
Yash621:main

Conversation

@Yash621

@Yash621 Yash621 commented Jan 11, 2022

Copy link
Copy Markdown

This PR is with respect to issue #2426

@alexr00 I have removed all the changes that are not related to fixing the bug,please review them.

let me know if any changes are required :)

@alexr00

alexr00 commented Jan 11, 2022

Copy link
Copy Markdown
Member

Looks like we lost some of the original styling (the arrow on the right of the select is gone):

image

The select now also has rounded corners, which isn't consistent with other selects in VS Code:
image

The colors look good though. Do you know if there's a way to fix the lost styling?

@Yash621

Yash621 commented Jan 11, 2022

Copy link
Copy Markdown
Author

@alexr00 sure , i would look into it and get back to you !

@alexr00 alexr00 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update! I'm seeing some style discrepancies, which I've left a comment on.

You can always push more commits to this PR and it will update, instead of creating a new PR each time :).

@Yash621

Yash621 commented Jan 20, 2022

Copy link
Copy Markdown
Author

@alexr00 I have tried to fix the styling but i am not getting like why the arrows have disappeared ,can you guide me on this ?

@alexr00

alexr00 commented Jan 21, 2022

Copy link
Copy Markdown
Member

@Yash621 I don't know why the arrows have disappeared off the top of my head so I don't have any guidance on this. I can do some digging into this later!

@Yash621

Yash621 commented Jan 21, 2022

Copy link
Copy Markdown
Author

@alexr00 Please see once ,and let me know as I have tried some approches but none is working so please let me know :)

@Yash621

Yash621 commented Feb 6, 2022

Copy link
Copy Markdown
Author

@alexr00 did you got any time to do some digging on why the arrows have disappeared ?

@alexr00

alexr00 commented Feb 7, 2022

Copy link
Copy Markdown
Member

@Yash621 not yet!

@Yash621 Yash621 closed this Feb 7, 2022
@Yash621 Yash621 reopened this Feb 7, 2022
@saifahn

saifahn commented Aug 2, 2022

Copy link
Copy Markdown

I believe it is due to -webkit-appearance: none;, which removes the glossy default Safari styling, but it also removes the arrows. You can recreate the arrows with CSS, though.

Is this still a problem looking to be solved? I'd be interested in working on it if so :)

@alexr00

alexr00 commented Aug 2, 2022

Copy link
Copy Markdown
Member

@saifahn, as far as I know #2426 is still a problem. If you want to take a crack at it please do!

@alexr00

alexr00 commented Dec 19, 2022

Copy link
Copy Markdown
Member

Closing due to lack of activity. @Yash621 if you're still planning to work on this please let me know and I'll reopen!

@alexr00 alexr00 closed this Dec 19, 2022
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.

3 participants