Amphtml: amp-selector[multiple] should display cursor: pointer on selectable options
In multiselect,
cursor:pointershould be on. Or more generically, if we have a meaningful action attached to the click (e.g. unselect for multiselect items), cursor should be pointer.
Related comment thread
Caveat: Unless item is disabled
caroqliu
All 8 comments
Hello. I am new and would love to contribute.
I would like to fix this issue, if no one is already on it.
THE-VR7
on 13 May 2020
Sure thing, thanks for your contribution!
caroqliu
on 13 May 2020
Hi, I am new here. If the issue is not already assigned I would like to work on it
AtharvaSune
on 17 May 2020
@THE-VR7 Let us know if you are still planning to contribute. Otherwise it sounds like @AtharvaSune will be happy to take this on. :) No rush either way.
caroqliu
on 18 May 2020
yeah sure @AtharvaSune can work on it, I am also new here and i am already working on another issue of this repository, so no problem
THE-VR7
on 18 May 2020
hey @caroqliu , from what i understood from the linked thread in the description, i need to make some changes in the extensions/amp-selector/0.1/amp-selector.css file so that amp-selector[multiple] displays pointer, right ?
AtharvaSune
on 19 May 2020
@AtharvaSune Yes that's right :)
caroqliu
on 19 May 2020
Hey @caroqliu just submitted the pr, could you have a look at it , :sweat_smile: if any further changes are required do tell
AtharvaSune
on 19 May 2020
Related issues
akshaylive
路
3Comments
aghassemi
路
3Comments
mkhatib
路
3Comments
torch2424
路
3Comments
samanthamorco
路
3Comments
Most helpful comment
yeah sure @AtharvaSune can work on it, I am also new here and i am already working on another issue of this repository, so no problem