Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New-UDSelect behaviour different across browsers #558

Closed
davegreen opened this issue Jan 16, 2019 · 3 comments
Closed

New-UDSelect behaviour different across browsers #558

davegreen opened this issue Jan 16, 2019 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@davegreen
Copy link

Using a New-UDSelect in a UDCard (instead of an input form) seems to generate some odd behaviour when viewing in Edge or IE. Things work roughly as I expect in Chrome, although values seem to get passed in quotes that appear from nowhere.

The returned data after the redirect appears as '[]' and makes the select non-functional if the user's browser is IE or Edge.

UD v2.2.1, Windows 10

New-UDCard -Title 'Example' -Content {
    New-UDSelect -Option {
        foreach ($option in @("one", "two")) {
            if ($option -eq $Session:Select) {
                New-UDSelectOption -Name $option -Value $option -Selected
            }
            else {
                New-UDSelectOption -Name $option -Value $option
            }
        }
    } -OnChange {
        $Session:Select = $eventdata
        Invoke-UDRedirect -Url "/$eventdata"
    }
}
@adamdriscoll adamdriscoll added the bug Something isn't working label Jan 16, 2019
@McAndersDK
Copy link
Contributor

This is the same as #574

@davegreen
Copy link
Author

@McAndersDK: Are you sure this is the same? This appears to still be an issue on the latest (v2.3.2)

adamdriscoll added a commit that referenced this issue Apr 12, 2019
@adamdriscoll adamdriscoll added this to the 2.4 milestone Apr 13, 2019
@adamdriscoll
Copy link
Member

This was a different issue. Now works in all browsers.

adamdriscoll added a commit that referenced this issue Apr 13, 2019
* Fixes #743 - Also, JS maps.

* Implements #720

* Implements #669

* Fixes #558

* Adds #483
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants