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

Fixes #16 #17

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fixes #16 #17

wants to merge 4 commits into from

Conversation

lopierra
Copy link
Contributor

@lopierra lopierra commented Jan 21, 2019

I recently learned how to use DataTables, so I thought I would revisit this. Fixes #16 .

@gdequeiroz
Copy link
Member

@lopierra I missed this. This looks great! The only issue I see is this:
image
The table and the map are overlapping.
Is there a way to change it?

…and change column widths to 6 and 6 (was previously 4 for DT and 8 for map). There is still overlap when browser window is very narrow, but at least it now works on my 13" laptop.
@lopierra
Copy link
Contributor Author

lopierra commented May 20, 2019

Thanks for testing! I didn't notice the overlap because my monitor at work is pretty wide. I resized the columns to 6 and 6 (previously 4 for table and 8 for map), so now it works on my small laptop as well. There is overlap if you make the browser window very narrow, but I'm not sure that can be avoided.

I think I see my commit here already, but let me know if there is something else I need to do. Thanks!

Fixes #16.

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.

Move tables to use the DataTables package
2 participants