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

ui: regression in single host or network subnet select (bootstrap select) #3148

Closed
fichtner opened this issue Jan 23, 2019 · 4 comments
Closed
Assignees
Labels
bug Production bug
Milestone

Comments

@fichtner
Copy link
Member

The subnet select will not give any options other than the default. See e.g. src/dst in firewall outbound NAT edit.

maybe due to #2848

via: https://forum.opnsense.org/index.php?topic=11274.0

@AdSchellevis
Copy link
Member

2af9202 fixes the behaviour, the layout however is still flawed.

@MichaelDeciso since your commit broke this, would you be so kind to fix the layout issues in the legacy code (for example in firewall_nat_edit.php).

In 18.7 this looks like this:
image

Now it looks like this:
image

It's likely a css issue in the supplied stylesheet #2939

@fichtner fichtner modified the milestones: 19.1, 19.7 Jan 23, 2019
fichtner pushed a commit that referenced this issue Jan 23, 2019
@MichaelDeciso
Copy link
Member

MichaelDeciso commented Jan 24, 2019

I have checked the Bootstrap documentation and the selectpicker CSS, and the problem is that the .input-group-btn class is not set here (but in most other places we use stuff like this, the class is set). This is compulsory according to the Bootstrap docs, and selectpicker 1.12 specifically looks for it. When I set the class, it looks and behaves correctly.

Will open a PR once I'm sure it works correctly and I haven't missed any other occurrences.

@MichaelDeciso
Copy link
Member

#3151

@AdSchellevis
Copy link
Member

@MichaelDeciso ok, looks good, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Production bug
Development

No branches or pull requests

3 participants