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

table made responsive #45

Merged
merged 17 commits into from
Feb 15, 2017
Merged

Conversation

satyammittal
Copy link
Member

@satyammittal satyammittal commented Feb 10, 2017

  • The table of latest tests isn't any good on mobile (maybe we should use a responsive table plugin?)
    In mobiles:
    fca817de-6601-420a-98c8-e89ba188b6b9
    In desktop:
    b994581d-bcd2-47e9-a536-e4bd772bcd44
    Table become list in small devices.
    p.s.: sorry for the random data placed inside the table.
    Responsiveness of the platform - pt. 2 #35

@canihavesomecoffee
Copy link
Member

Please update the PR to fix the aligment issues, and the "broken" html near the table element.

@satyammittal
Copy link
Member Author

can you please specify what are the alignment issues?

@satyammittal
Copy link
Member Author

screen shot 2017-02-15 at 16 37 19

<th title="Did the test suite finish or error out?">Completed</th>
{% if tests|length > 0 %}
<div id="no-more-tables">
<table class="col-md-12 table-bordered table-striped table-condensed cf">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alignment here.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@canihavesomecoffee
Copy link
Member

That's looking all good, but could you now fix the conflict between the master and your PR in app.css?

@satyammittal
Copy link
Member Author

ok i will fix the conflict

@madan96
Copy link
Contributor

madan96 commented Feb 15, 2017

@satyammittal Try fetch and rebase instead of pull. It results in a cleaner commit history :) . Also, I think you can squash the many of the commits.

@canihavesomecoffee
Copy link
Member

I definitely am going to squash that :p

@canihavesomecoffee canihavesomecoffee merged commit f8331ad into CCExtractor:master Feb 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants