Minor updates to plot.ly/api #1057

jdamiba · 2018-08-27T20:31:33Z

closes https://github.com/plotly/streambed/issues/11343

changed API Libraries to Open Source Graphing Libraries throughout.

added a box for dash and for REST API.

Removed API Protocol and Excel menus.

Changed Plotly Workshop to DASH DAQ.

changed footers to match plot.ly : closes #1055

jdamiba · 2018-08-27T20:32:16Z

Outstanding question: which images should I use for the REST API box?

jackparmer · 2018-08-27T22:10:03Z

This one looks pretty good in terms of matching the others and being clean:

https://cdn-images-1.medium.com/max/1200/1*uHzooF1EtgcKn9_XiSST4w.png

Kully · 2018-08-29T15:49:36Z

Going to give this a review now

Kully · 2018-08-29T16:43:47Z

The REST icon is too small. Can you make it bigger so that it matches the size of the other icons around it?

Also replace the dash icon you are using with
https://github.com/plotly/design-assets/blob/master/logos/dash/images/dash-logo-avatar.png.

It is the standard avatar for dash and consistent with the icon used in the page it links to.

Kully · 2018-08-29T16:51:21Z

Not blocking 🚫 , but I think putting Python, R and Others sections on one line would look better in my opinion. All that extra white space on the Others section just looks weird to me.

Kully · 2018-08-29T17:07:06Z

_posts/python/3d/2017-02-24-python-3d-index.html

@@ -19,7 +19,7 @@
 		<div class="--title">
 			<div class="--category-img"><img src="https://plot.ly/gh-pages/documentation/static/images/python-small.png" alt=""></div>
 			<div class="--body">
-				<h1>Plotly Python Library 3D Charts</h1>
+				<h1>Plotly Python Open Source Library 3D Charts</h1>


should this not be Open Source Graphing Library to be consistent?

Kully · 2018-08-29T17:07:15Z

_posts/python/basic/2017-02-24-python-basic-index.html

@@ -19,7 +19,7 @@
 		<div class="--title">
 			<div class="--category-img"><img src="https://plot.ly/gh-pages/documentation/static/images/python-small.png" alt=""></div>
 			<div class="--body">
-				<h1>Plotly Python Library Basic Charts</h1>
+				<h1>Plotly Python Open Source Library Basic Charts</h1>


same here as ^

Kully · 2018-08-29T17:08:28Z

_posts/python/fundamentals/2017-02-24-python-fundamentals-index.html

@@ -19,7 +19,7 @@
 		<div class="--title">
 			<div class="--category-img"><img src="https://plot.ly/gh-pages/documentation/static/images/python-small.png" alt=""></div>
 			<div class="--body">
-				<h1>Plotly Python Library Fundamentals</h1>
+				<h1>Plotly Python Open Source Library Fundamentals</h1>


same here: ...Graphing Library

Kully · 2018-08-29T17:09:09Z

_posts/python/fundamentals/config/2015-06-30-config_opts.html

@@ -31,7 +31,7 @@ <h4 id="New-to-Plotly?">New to Plotly?<a class="anchor-link" href="#New-to-Plotl
 </div>
 <div class="inner_cell">
 <div class="text_cell_render border-box-sizing rendered_html">
-<h4 id="Online-Configuration-Options">Online Configuration Options<a class="anchor-link" href="#Online-Configuration-Options">&#194;&#182;</a></h4><p>Config options set via our API libraries are overridden on graphs hosted on plot.ly (i.e. when working online).
+<h4 id="Online-Configuration-Options">Online Configuration Options<a class="anchor-link" href="#Online-Configuration-Options">&#194;&#182;</a></h4><p>Config options set via our Open Source Graphing libraries are overridden on graphs hosted on plot.ly (i.e. when working online).


capitalize libraries -> Libraries for consistency

Kully · 2018-08-29T17:09:31Z

_posts/python/scientific/2017-02-24-python-scientific-index.html

@@ -19,7 +19,7 @@
 		<div class="--title">
 			<div class="--category-img"><img src="https://plot.ly/gh-pages/documentation/static/images/python-small.png" alt=""></div>
 			<div class="--body">
-				<h1>Plotly Python Library Scientific Charts</h1>
+				<h1>Plotly Python Open Source Library Scientific Charts</h1>


graphing library

Kully · 2018-08-29T17:10:22Z

all_static/images/Plotly-Feed.html

@@ -12381,7 +12381,7 @@ <h4 class="delta heading text--left">The Plotly community</h4>
                </div><!--
                --><div class="grid__item one-half">
                    <h4 class="delta heading text--left">Programmer? We've got you covered.</h4>
-                    See our <a href="https://plot.ly/api/" target="_self" class="link link--impt epsilon">API libraries</a><br>
+                    See our <a href="https://plot.ly/api/" target="_self" class="link link--impt epsilon">Open Source Graphing libraries</a><br>


capital L on libraries

Kully · 2018-08-29T17:10:37Z

_posts/python/statistical/2017-02-24-python-statistic-index.html

@@ -19,7 +19,7 @@
 		<div class="--title">
 			<div class="--category-img"><img src="https://plot.ly/gh-pages/documentation/static/images/python-small.png" alt=""></div>
 			<div class="--body">
-				<h1>Plotly Python Library Statistical Charts</h1>
+				<h1> Library Statistical Charts</h1>


why are you removing Plotly Python here?

Kully · 2018-08-29T17:10:50Z

all_static/images/Plotly-feed2.html

@@ -12381,7 +12381,7 @@ <h4 class="delta heading text--left">The Plotly community</h4>
                </div><!--
                --><div class="grid__item one-half">
                    <h4 class="delta heading text--left">Programmer? We've got you covered.</h4>
-                    See our <a href="https://plot.ly/api/" target="_self" class="link link--impt epsilon">API libraries</a><br>
+                    See our <a href="https://plot.ly/api/" target="_self" class="link link--impt epsilon">Open Source Graphing libraries</a><br>


Kully · 2018-08-29T17:11:00Z

all_static/images/Plotly.html

@@ -12381,7 +12381,7 @@ <h4 class="delta heading text--left">The Plotly community</h4>
                </div><!--
                --><div class="grid__item one-half">
                    <h4 class="delta heading text--left">Programmer? We've got you covered.</h4>
-                    See our <a href="https://plot.ly/api/" target="_self" class="link link--impt epsilon">API libraries</a><br>
+                    See our <a href="https://plot.ly/api/" target="_self" class="link link--impt epsilon">Open Source Graphing libraries</a><br>


Kully · 2018-08-29T17:12:27Z

_posts/pandas/2015-07-19-pandas_index.html

@@ -12,7 +12,7 @@
 		<div class="--title">
 			<div class="--category-img"><img src="https://plot.ly/gh-pages/documentation/static/images/pandas-small.png" alt=""></div>
 			<div class="--body">
-				<h1>Plotly Python Library for Pandas</h1>
+				<h1>Plotly Python Open Source Library for Pandas</h1>


jdamiba · 2018-08-29T19:07:17Z

I standardized the use of "Open Source Graphing Libraries" throughout the site.

In terms of images, does anyone know how i can host images on images.plot.ly? That's where the other images for the homepage are being served from.

Kully · 2018-08-30T13:48:43Z

@jdamiba give me a ping when you've addressed/responded to all my comments. Thanks 😄

nicolaskruchten · 2018-08-30T13:53:21Z

Once you deploy https://github.com/plotly/plotly.github.io/pull/366 you can use that image ;)

Resolving conflicts with source-design-merge

cldougl · 2018-09-04T16:33:28Z

I'm a bit unsure about adding the rest library in that spot as it is not open-sourced

jdamiba · 2018-09-04T17:20:00Z

I removed the REST API box. I like the way it looks now a lot since Dash gets a level all to itself.

cldougl

💃

cldougl · 2018-09-04T20:45:45Z

We may want to consider making the icons per row 3, 2 rather than 4, 1 in the above screenshot though that can be done in a follow up pr

nicolaskruchten · 2018-09-04T20:48:02Z

+1 to the 3/2 vs 4/1 per row.

Other thoughts:

We want to move away from "Download free and open-source clients for:"
We need to be clear that Dash isn't like the others... it's not another "Open Source Graphing" library per se... App Development Library?

nicolaskruchten · 2018-09-04T20:55:18Z

OK, let's ship it 💃

change API to Open Source throughout

cbf4d42

jdamiba requested review from jackparmer and priyatharsan August 27, 2018 20:31

Joe Damiba added 2 commits August 27, 2018 18:21

changed image for REST API

62c77fb

added oxford comma to subheader

ca6181c

Kully reviewed Aug 29, 2018

View reviewed changes

standardized Open Source Graphing Libraries throughout site

8f467de

changed dash logo

3768722

cldougl requested review from cldougl and removed request for jackparmer and priyatharsan September 4, 2018 15:23

cldougl added the ReviewReady This PR is ready for review label Sep 4, 2018

Merge branch 'source-design-merge' into relabel-api

173c98b

Resolving conflicts with source-design-merge

added scripts back to footer

8c251e9

removed REST API box

5320b24

cldougl approved these changes Sep 4, 2018

View reviewed changes

changed subtitle

5cd920a

jdamiba merged commit 0757771 into source-design-merge Sep 4, 2018

jdamiba mentioned this pull request Sep 5, 2018

Align footer with plot.ly #1055

Closed

cldougl mentioned this pull request Sep 5, 2018

https://plot.ly/api/ edit suggestions (round 2) #1075

Closed

2 tasks

Uh oh!

Minor updates to plot.ly/api #1057

Minor updates to plot.ly/api #1057

Uh oh!

Conversation

jdamiba commented Aug 27, 2018 • edited by nicolaskruchten Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jdamiba commented Aug 27, 2018

Uh oh!

jackparmer commented Aug 27, 2018

Uh oh!

Kully commented Aug 29, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Kully commented Aug 29, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Kully commented Aug 29, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Kully Aug 29, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jdamiba commented Aug 29, 2018

Uh oh!

Kully commented Aug 30, 2018

Uh oh!

nicolaskruchten commented Aug 30, 2018

Uh oh!

cldougl commented Sep 4, 2018

Uh oh!

jdamiba commented Sep 4, 2018

Uh oh!

cldougl left a comment

Choose a reason for hiding this comment

Uh oh!

cldougl commented Sep 4, 2018

Uh oh!

nicolaskruchten commented Sep 4, 2018

Uh oh!

nicolaskruchten commented Sep 4, 2018

Uh oh!

Uh oh!

jdamiba commented Aug 27, 2018 •

edited by nicolaskruchten

Loading

Kully commented Aug 29, 2018 •

edited

Loading

Kully commented Aug 29, 2018 •

edited

Loading

Kully Aug 29, 2018 •

edited

Loading