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

Dividing the tags page to make it two-column and add graph in second column #5100

Merged
merged 2 commits into from
Mar 19, 2019

Conversation

CleverFool77
Copy link
Member

@CleverFool77 CleverFool77 commented Mar 16, 2019

Fixes #5090 first part and second part (<=== Add issue number here)

Description

For building the new tags page, the first step was to divide the tags page into two column.
And second was to add an iframe to second column.

Screenshot from 2019-03-19 09-22-13

  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
  • code is in uniquely-named feature branch and has no merge conflicts 📁
  • PR is descriptively titled 📑
  • screenshots/GIFs are attached 📎 in case of UI updation
  • ask @publiclab/reviewers for help, in a comment below

Thanks!

@plotsbot
Copy link
Collaborator

plotsbot commented Mar 16, 2019

1 Message
📖 @CleverFool77 Thank you for your pull request! I’m here to help with some tips and recommendations. Please take a look at the list provided and help us review and accept your contribution! And don’t be discouraged if you see errors – we’re here to help.

Generated by 🚫 Danger

@CleverFool77
Copy link
Member Author

CleverFool77 commented Mar 18, 2019

@jywarren @gauravano Can I have some insight regarding this as I need to make a new PR for stats graph in second column. But before that I need this PR to be required first. 😅

@jywarren
Copy link
Member

This looks awesome! Shall we try to add the <iframe style="border:0;width:100%;height:500px;" src="/stats/graph?limit=50"></iframe> code in the second column, so that we don't publish code that has a big empty space? Thank you!!!!

@CleverFool77 CleverFool77 changed the title Dividing the tags page to make it two-column Dividing the tags page to make it two-column and add graph in second column Mar 19, 2019
@CleverFool77
Copy link
Member Author

@jywarren @gauravano I did as you suggested and added the second part of issue in this PR itself.
Thanks :)

@grvsachdeva
Copy link
Member

Adding ready on basis of @jywarren review. Thanks!

@jywarren jywarren merged commit aabcc20 into publiclab:master Mar 19, 2019
@jywarren
Copy link
Member

Awesome!!!

cesswairimu pushed a commit to cesswairimu/plots2 that referenced this pull request Mar 21, 2019
…column (publiclab#5100)

* Dividing the tags page to make it two-column

* add stats graph in second column
icarito pushed a commit to icarito/plots2 that referenced this pull request Apr 9, 2019
…column (publiclab#5100)

* Dividing the tags page to make it two-column

* add stats graph in second column
SrinandanPai pushed a commit to SrinandanPai/plots2 that referenced this pull request May 5, 2019
…column (publiclab#5100)

* Dividing the tags page to make it two-column

* add stats graph in second column
@CleverFool77 CleverFool77 deleted the two-column branch June 28, 2019 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants