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

Inputbox set minimum width entry field #234

Merged
merged 1 commit into from Jul 28, 2017

Conversation

Projects
None yet
2 participants
@JamesBremner
Contributor

JamesBremner commented Jul 27, 2017

V1.5.4 increased the default minimum width of the entry fields in inputbox to a sensible value of 100 pixels. However, we sometimes need even wider fields than this, but it would be weird to make the default any wider than 100 pixels. This add a setter to increase the width when needed.

@JamesBremner

This comment has been minimized.

Show comment
Hide comment
@JamesBremner

JamesBremner Jul 28, 2017

Contributor

Usage example:

    form fm;

    // construct text input the expects long inputs
    inputbox::text test(
        "very very very very long prompt string",
        "very long default");
        
    // construct inputbox that provides especially wide input fields
    inputbox inbox(fm,"Enter");
    inbox.min_width_entry_field( 250 );
    
    //show 'em what we have
    if (inbox.show(test) )
    {
    }

image

Contributor

JamesBremner commented Jul 28, 2017

Usage example:

    form fm;

    // construct text input the expects long inputs
    inputbox::text test(
        "very very very very long prompt string",
        "very long default");
        
    // construct inputbox that provides especially wide input fields
    inputbox inbox(fm,"Enter");
    inbox.min_width_entry_field( 250 );
    
    //show 'em what we have
    if (inbox.show(test) )
    {
    }

image

@cnjinhao cnjinhao merged commit 953450a into cnjinhao:develop Jul 28, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@cnjinhao

This comment has been minimized.

Show comment
Hide comment
@cnjinhao

cnjinhao Jul 28, 2017

Owner

Thanks for your PR!

Owner

cnjinhao commented Jul 28, 2017

Thanks for your PR!

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