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

Form rendering is off within a summary section #1696

Closed
yatesco opened this issue Mar 2, 2018 · 3 comments · Fixed by tghelere/valorize-vidas#6 · May be fixed by devopsred/gaia#6, anikets43/angular-ngrx-socket-frontend#4 or jwlh/WDI_GROUP_PROJECT#2

Comments

@yatesco
Copy link

yatesco commented Mar 2, 2018

This is about Bulma.

Overview of the problem

Description

Forms rendered inside a <details> inside a <nav> don't render correctly. If I remove the <details> and <summary> tag then the form renders correctly. Specifically, the broken form extends past the right hand side of the containing div.

In the screenshot you can see both a working and non-working form:

screen shot 2018-03-02 at 21 57 18

The broken form is configured:

<nav className="panel">
    <details open>
        <summary className="panel-heading">Forms look good inside collapsible panels</summary>
        <p className="panel-header">
            <div className="panel-block">
                <form>...</.form>
            </div>
        </p>
    </details>
</nav>

The working form is configured:

<nav className="panel">
    <p className="panel-heading">Forms look good inside static panels</p>
    <p className="panel-header">
        <div className="panel-block">
            <form>...</.form>
        </div>
    </p>
</nav>
@yatesco
Copy link
Author

yatesco commented Mar 2, 2018

The original issue shows <input type="text"..., however the rendering is much worse for select items:
screen shot 2018-03-02 at 22 26 33

aelharrak added a commit to aelharrak/bulma that referenced this issue Mar 7, 2018
@aelharrak
Copy link

Hello @yatesco : i was create an new pull request for this issue :)

BR,
Amine

@dailycommit
Copy link
Contributor

@aelharrak Thank you for the fixes but you should also add
box-sizing: border-box to .controlclass

see: https://www.dropbox.com/s/ribss13dei8l0n8/bulma-box-sizing.mp4?dl=0

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