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

Firewall Rules: Text -> Combobox for VPC, VPC Subnet, and Instance fields #2065

Closed
askfongjojo opened this issue Mar 13, 2024 · 2 comments · Fixed by #2296
Closed

Firewall Rules: Text -> Combobox for VPC, VPC Subnet, and Instance fields #2065

askfongjojo opened this issue Mar 13, 2024 · 2 comments · Fixed by #2296
Assignees
Milestone

Comments

@askfongjojo
Copy link

askfongjojo commented Mar 13, 2024

When creating a firewall rule, user has the options of specifying the source (aka "Host filters") and target using VPC, VPC subnet, or a specific instance as the scope. The name of these objects is currently captured in a free-form textbox which may be prone to typos and is just harder to use.

We already have the target/host name field label changed dynamically based on the type field. We probably want to go one step further to make the name field dynamically display the candidates of that target/host type, similar to what we do for Create Disk:
image

Note: "Picklist" is mentioned in a general sense here. It's likely that a picklist of instances may not be practical since there could be many of them. Perhaps we need a different mechanism for listing/specifying an instance name. For IP and IP subnet, textbox may still be the most appropriate input format.

@askfongjojo askfongjojo added this to the 8 milestone Mar 13, 2024
@david-crespo david-crespo removed this from the 8 milestone Apr 15, 2024
@david-crespo david-crespo changed the title Firewall Rules: Provide picklists for VPC, VPC subnet, and Instance when they are specified as the source or target Firewall Rules: Text -> Listbox/Combobox for VPC, VPC Subnet, and Instance fields Apr 15, 2024
@david-crespo david-crespo changed the title Firewall Rules: Text -> Listbox/Combobox for VPC, VPC Subnet, and Instance fields Firewall Rules: Text -> Combobox for VPC, VPC Subnet, and Instance fields Apr 15, 2024
@david-crespo
Copy link
Collaborator

https://www.w3.org/WAI/ARIA/apg/patterns/combobox/

A combobox is an input widget that has an associated popup. The popup enables users to choose a value for the input from a collection. The popup may be a listbox, grid, tree, or dialog.

In some implementations, the popup presents allowed values, while in other implementations, the popup presents suggested values.

I suspect what we want here is the latter: an input that remains fundamentally a free-form text input with some help from a filterable list if you want it. I don't think we'd want to constrain the user to only refer to VPCs, subnets, or instances that currently exist. I believe this API is designed so that you can create the rule first, if you want, and then create the instance that matches the rule.

@charliepark
Copy link
Contributor

charliepark commented May 8, 2024

Check #1942 to see if it fixes this unrelated, I think

@david-crespo david-crespo added this to the 9 milestone May 10, 2024
@morlandi7 morlandi7 assigned charliepark and unassigned charliepark Jun 24, 2024
@david-crespo david-crespo modified the milestones: 9, 10 Jul 9, 2024
@morlandi7 morlandi7 modified the milestones: 10, 11 Aug 26, 2024
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 a pull request may close this issue.

4 participants