Skip to content

Conversation

@shimsangjoon
Copy link
Contributor

adding ability to remove previous players from personal opening search history.

@kraktus
Copy link
Member

kraktus commented Sep 14, 2023

hey, thanks for the contribution! A screenshot for a UI change is always welcomed.

@shimsangjoon
Copy link
Contributor Author

Of course. I'm not proud of the UI I did so perhaps someone else can improove that part. However I believe the functionality is as requested.

Screenshot 2023-09-14 at 17 54 40

@shimsangjoon
Copy link
Contributor Author

Minor improvement using opacity change for visual confirmation instead of crosshair cursor

Screenshot 2023-09-16 at 12 20 37

@Siderite
Copy link

Siderite commented Oct 3, 2023

Just a slight informational message, this is how it looks with the LiChess Tools extension:
image

While I can't call myself any kind of UI designer, I believe a small white x next to the name might be a lot easier to mistakenly click. May I suggest you make it more visible (different color) and perhaps change its position so it's not in line with the text?

I think I like your design better than mine, but I feel it's a bit more dangerous, if that makes sense.

@kraktus
Copy link
Member

kraktus commented Oct 3, 2023

There’s a PR with a more refined design here #13518

@ornicar
Copy link
Collaborator

ornicar commented Oct 3, 2023

merged #13518 instead

@ornicar ornicar closed this Oct 3, 2023
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.

4 participants