vscode-pull-request-github icon indicating copy to clipboard operation
vscode-pull-request-github copied to clipboard

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

Open Yash621 opened this issue 3 years ago • 9 comments

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 :)

Yash621 avatar Jan 11 '22 10:01 Yash621

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?

alexr00 avatar Jan 11 '22 12:01 alexr00

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

Yash621 avatar Jan 11 '22 12:01 Yash621

@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 ?

Yash621 avatar Jan 20 '22 11:01 Yash621

@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!

alexr00 avatar Jan 21 '22 09:01 alexr00

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

Yash621 avatar Jan 21 '22 15:01 Yash621

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

Yash621 avatar Feb 06 '22 18:02 Yash621

@Yash621 not yet!

alexr00 avatar Feb 07 '22 14:02 alexr00

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 :)

saifahn avatar Aug 02 '22 13:08 saifahn

@saifahn, as far as I know https://github.com/microsoft/vscode-pull-request-github/issues/2426 is still a problem. If you want to take a crack at it please do!

alexr00 avatar Aug 02 '22 13:08 alexr00

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 avatar Dec 19 '22 16:12 alexr00