Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Fix unable to select items in a multi-picklist #953 #1001

Merged
merged 2 commits into from
Mar 14, 2019
Merged

Conversation

MortimerGoro
Copy link
Contributor

No description provided.

@cvan
Copy link
Contributor

cvan commented Mar 13, 2019

Copy link
Contributor

@cvan cvan left a comment

Choose a reason for hiding this comment

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

👍 Nice work, @MortimerGoro. This fixes the original issue. There are a few items I wanted to still address.

I noticed a few inconsistencies and bugs, primarily an issue with the <option>s not being properly highlighted upon being checked (selected) in a <select multiple> field. If this is a GeckoView issue, let's file there.


In FxR, in a rendered <select multiple> on the page, when any values (e.g., the first two values) are selected, the rows are not properly highlighted as they are in Firefox for desktop:

image

Notice that in FxR only the first row is ever "highlighted" (with the faint-blue focus ring):

image

Compare with Oculus Browser:

  1. Load <select multiple> test on https://webxr.sh/dropdowns#multiple.

  2. Notice the default text will read Cherry (but if you modify the value and then reload the page, it will continue to read 2 Selected).

    image
    image


  1. In Oculus Browser, Firefox Reality, and Firefox for desktop, scroll to https://webxr.sh/dropdowns#multiple.
  2. Press the <select multiple> field, and press the rows such that four items are selected.
  3. Press the checkmark icon to submit the changes.
  4. Notice the text changes from -- Please choose one or more -- to 4 Selected.

In Oculus Browser:

image

in FxR:

image

vs. in Firefox for desktop:

image


also, I filed #1007 for an issue I noticed with <option disabled> (example):

image

@MortimerGoro
Copy link
Contributor Author

MortimerGoro commented Mar 14, 2019

@cvan can you reproduce the highlight issue consistently? I tried many times and only was able to reproduce it once. At the same time #1007 happened too, but not on the others attemps. So I think they are related and it's a GV bug, because the values in the prompt are kept selected correctly.

@bluemarvin bluemarvin merged commit f76a9d8 into master Mar 14, 2019
@ghost ghost removed the in progress label Mar 14, 2019
@bluemarvin
Copy link
Contributor

@cvan trying to roll a release. Any additional issues can be handled in follow up.

@cvan
Copy link
Contributor

cvan commented Mar 14, 2019

@cvan can you reproduce the highlight issue consistently? I tried many times and only was able to reproduce it once. At the same time #1007 happened too, but not on the others attemps. So I think they are related and it's a GV bug, because the values in the prompt are kept selected correctly.

I can consistently reproduce on Oculus Go and Pico G2 (WaveVR) the highlight issue with <option selected> in <select multiple> issue. I've filed issue #1013.

I can also consistently reproduce (#1007 (comment)) the bug with the hidden text in <option disabled>. thanks for filing GeckoView bug 1535275 to track.

@cvan cvan deleted the listview branch March 14, 2019 22:16
@cvan cvan added this to the v1.1.3 milestone Mar 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants