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.

Sign up for GitHub

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

Jump to bottom

r-table breaks input-field and formal-field #63

Closed
sultanDMTR opened this issue Feb 8, 2017 · 3 comments
Closed

r-table breaks input-field and formal-field #63

sultanDMTR opened this issue Feb 8, 2017 · 3 comments
Assignees
@ceremcem
Labels
bug or major UX

Comments

@sultanDMTR
Copy link
Collaborator

sultanDMTR commented Feb 8, 2017 •
edited

This is the expected view of input-field component from the showcase:
image

However if input-field is placed in data-table's edit-form, it looks as follows:
image

Reproduce

Issue can be reproduced by the following PR: #62

  1. Navigate to data-table section in http://localhost:4001/showcase.html
  2. Click a row to open view-form
  3. Click pencil checkbox to open edit-form an d see the screenshot above
@sultanDMTR sultanDMTR changed the title FIXME: rwd-table causes unexpected behaviour on other tables Data-table breaks input-field Feb 8, 2017
@ozanmuyes ozanmuyes mentioned this issue Feb 8, 2017
Closed
@sultanDMTR sultanDMTR changed the title Data-table breaks input-field Data-table breaks input-field and formal-field Feb 9, 2017
@ceremcem ceremcem changed the title Data-table breaks input-field and formal-field r-table breaks input-field and formal-field Feb 9, 2017
@ceremcem ceremcem assigned ceremcem and unassigned ozanmuyes Feb 9, 2017
@ceremcem
Copy link
Contributor

ceremcem commented Feb 9, 2017

As of #68 , this problem is related with r-table's css.

@sultanDMTR
Copy link
Collaborator Author

Previous view of data-table component is like:

image

Right now it looks like:

image

@ceremcem ceremcem reopened this Feb 9, 2017
@ozanmuyes
Copy link
Contributor

ozanmuyes commented Feb 9, 2017 •
edited

@ceremcem I insist you to merge the PR #64 while we find a proper solution than a work-around.

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

@ceremcem ceremcem

Labels
bug or major UX
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ozanmuyes @ceremcem @sultanDMTR