creativetimofficial / ct-paper-dashboard-pro-react

17 stars 6 forks source link

Modal headers & footer layout issues #6

Closed tylerdavis closed 4 years ago

tylerdavis commented 5 years ago

!!! IF YOU DO NOT USE THIS ISSUES TEAMPLATE, YOUR ISSUE IS LIABLE TO BEING IGNORED BY US

Hey there. I just purchased a business license for this theme and have run into a small issue I was hoping you might be able to address. If I have left anything important out, please let me know and I will update this issue accordingly.

Thank you very much

Prerequisites

Please answer the following questions for yourself before submitting an issue.

Expected Behavior

Modal headers: The default bootstrap modal layout should render the documented styles correctly, honoring the vertical alignment in the examples.

Modal footers: The right padding on the modal footers needs to be adjusted.

Current Behavior

I use the data-confirm-modal gem for rails confirmations, which renders the default modal. Additionally I have several modals that I built prior to beginning to implement your theme that are built to the bootstrap 4 spec. The header content does not align correctly. The close button ends up below the header title in all cases.

Screenshot 2019-06-09 09 25 07

I see that certain examples in your documentation also behave like this.

Screenshot 2019-06-09 09 28 59

Failure Information (for bugs)

Using the documented modal layouts will produce this bug.

Steps to Reproduce

See your documentation on modals for steps to reproduce.

Context

My application is built on ruby on rails. Additionally, the frontend is a mix of vanilla rails / bootstrap / erb and reactstrap.

Failure Logs

Please include any relevant log snippets or files here.

einazare commented 5 years ago

Hello there, @tylerdavis ,

Thank you for your interest in working with our products. You can go ahead and add the following css:

.modal-header {
    display: flex !important;
}

Best, Manu