Re: [racket-dev] [plt] Push #27967: master branch updated

2013-12-20 Thread Robby Findler
Community seems ok to me.

Robby

On Friday, December 20, 2013, wrote:

> jay has updated `master' from 438942c059 to 9012f7b3d6.
>   http://git.racket-lang.org/plt/438942c059..9012f7b3d6
>
> =[ 5 Commits ]==
> Directory summary:
>   85.5% pkgs/plt-services/meta/pkg-index/official/static/
>7.0% pkgs/plt-services/meta/web/stubs/
>3.8% pkgs/
>3.5% racket/collects/pkg/
>
> ~~
>
> abc8b30 Jay McCarthy > 2013-12-20 15:40
> :
> | Fix typo on pkg site
> :
>   M pkgs/plt-services/meta/pkg-index/official/static/index.html | 2 +-
>
> ~~
>
> 6c4650e Jay McCarthy > 2013-12-20 15:47
> :
> | Remove secret information from pkg error messages
> :
>   M racket/collects/pkg/util.rkt | 4 +++-
>
> ~~
>
> 7faab4d Jay McCarthy > 2013-12-20 15:53
> :
> | Fix PR14216
> :
>   M racket/collects/pkg/lib.rkt | 5 +++--
>
> ~~
>
> 490e21f Jay McCarthy > 2013-12-20 16:20
> :
> | Use Racket navbar on pkgs. and have navbar link to pkgs.
> |
> | Open question: Where does link to Planet go? Sam's new design has a
> natural place, but on the old site... it's not clear, community?
> :
>   M .../meta/pkg-index/official/static/index.html |  2 +
>   M .../meta/pkg-index/official/static/style.css  | 49
> 
>   M pkgs/plt-services/meta/web/all.rkt|  2 +-
>   C pkgs/plt-services/meta/{web/common =>
> pkg-index/official/static}/logo.png (100%)
>   M pkgs/plt-services/meta/web/config.rkt |  1 +
>   M pkgs/plt-services/meta/web/stubs/all.rkt  |  5 +-
>   C pkgs/plt-services/meta/web/stubs/{planet.rkt => packages.rkt} (75%)
>
> ~~
>
> 9012f7b Jay McCarthy > 2013-12-20 16:29
> :
> | Adding suggestion about 1.0 package versions
> :
>   M .../racket-doc/pkg/scribblings/getting-started.scrbl | 5 -
>
> =[ Overall Diff ]===
>
> pkgs/plt-services/meta/pkg-index/official/static/index.html
> ~~~
> --- OLD/pkgs/plt-services/meta/pkg-index/official/static/index.html
> +++ NEW/pkgs/plt-services/meta/pkg-index/official/static/index.html
> @@ -10,6 +10,8 @@
>  
>
>
> + cellpadding="0" cellspacing="0" width="100%">http://racket-lang.org/";>(( style="font-size: 60px; vertical-align: middle;">( class="navtitle" style="font-size: 40px; vertical-align:
> middle;">  style="vertical-align: middle; margin: 13px 0.25em 0 0; border: 0;"> class="navtitle" style="font-size: 80px; vertical-align:
> middle;">Racket  style="font-size: 60px; vertical-align: middle;">) class="navtitle" style="font-size: 80px; vertical-align:
> middle;">)) href="http://racket-lang.org/help.html";>Need
> Help? width="100%"> class="navlink"> href="http://racket-lang.org/";>About class="navlinkcell">http://racket-lang.org/download/";>Download class="navlinkcell">http://docs.racket-lang.org/";>Documentation class="navlinkcell"> href="/">Packages class="navitem">http://racket-lang.org/community.html";>Community class="navlinkcell">http://racket-lang.org/learning.html";>!
>  Learning!
>  
> +
>  
>Packages
>
> @@ -55,7 +57,7 @@
>  Description: id="pi_description">
>  Tags:
>   id="pi_add_tag_text" class="text ui-widget-content ui-corner-all" /> id="pi_add_tag_button">Add Tag
> -Versions Exceptions id="pi_versions">
> +Version Exceptions id="pi_versions">
>  Version:
> Source:  id="pi_add_version_source_text" class="text ui-widget-content
> ui-corner-all" />Add Version
> Exception
>  Dependencies id="pi_dependencies">
>  Conflicts id="pi_conflicts">
>
> pkgs/plt-services/meta/pkg-index/official/static/style.css
> ~~
> --- OLD/pkgs/plt-services/meta/pkg-index/official/static/style.css
> +++ NEW/pkgs/plt-services/meta/pkg-index/official/static/style.css
> @@ -154,3 +154,52 @@ a.possible {
>  tr#pi_delete_row td {
>  text-align: center;
>  }
> +
> +/* racket nav */
> +.racketnav {
> +  background-color: #00;
> +  color: #ff;
> +  margin-bottom: 1em;
> +  padding: 0.5em 0em;
> +  white-space: nowrap;
> +}
> +.racketnav a {
> +  color: #ff;
> +  text-decoration: none;
> +}
> +.racketnav .navcontent {
> +  margin-left: auto;
> +  margin-right: auto;
> +  width: 45em;
> +  font-family: Optima, Arial, Verdana, Helvetica, sans-serif;
> +}
> +.racketnav .navtitle {
> +  font-size: xx-large;
> +  font-weight: bold;
> +}
> +.racketnav .navitem {
> +  text-decoration: none;
> +  font-size: 88%;
> +}
> +.racketnav .navlink a {
> +  padding: 0em 1em;
> +}
> +.racketnav .navcurlink a {
> +  padding: 0em 1em;
> +  background-color: #55;
> +}
> +.racketnav .navlinka:hover,
> +.racketnav .navcurlink a:hover {
> +  background-color: #88;
> +}
> +.racketnav .navlinkcell {
> +  text-align: center;
> +}
> +.racketnav .helpiconcell {
> +  text-align: right;
> +  ve

Re: [racket-dev] [plt] Push #27967: master branch updated

2013-12-20 Thread Sam Tobin-Hochstadt
On Fri, Dec 20, 2013 at 6:35 PM, Jay McCarthy  wrote:
> Where should we put the link to Planet?
>
> - Main page
> - New place
> - Community
> - Keep it on the navbar with packages
>
> I can see good and bad things with most.

I think "Main Page" & "Community" both is the right choice.  I also
should say that the new web page work is basically done, but I don't
really have a way to test it (and I don't want to break things right
before the release).

Sam
_
  Racket Developers list:
  http://lists.racket-lang.org/dev


Re: [racket-dev] [plt] Push #27967: master branch updated

2013-12-20 Thread Jay McCarthy
Where should we put the link to Planet?

- Main page
- New place
- Community
- Keep it on the navbar with packages

I can see good and bad things with most.

On Fri, Dec 20, 2013 at 4:32 PM,   wrote:
> jay has updated `master' from 438942c059 to 9012f7b3d6.
>   http://git.racket-lang.org/plt/438942c059..9012f7b3d6
>
> =[ 5 Commits ]==
> Directory summary:
>   85.5% pkgs/plt-services/meta/pkg-index/official/static/
>7.0% pkgs/plt-services/meta/web/stubs/
>3.8% pkgs/
>3.5% racket/collects/pkg/
>
> ~~
>
> abc8b30 Jay McCarthy  2013-12-20 15:40
> :
> | Fix typo on pkg site
> :
>   M pkgs/plt-services/meta/pkg-index/official/static/index.html | 2 +-
>
> ~~
>
> 6c4650e Jay McCarthy  2013-12-20 15:47
> :
> | Remove secret information from pkg error messages
> :
>   M racket/collects/pkg/util.rkt | 4 +++-
>
> ~~
>
> 7faab4d Jay McCarthy  2013-12-20 15:53
> :
> | Fix PR14216
> :
>   M racket/collects/pkg/lib.rkt | 5 +++--
>
> ~~
>
> 490e21f Jay McCarthy  2013-12-20 16:20
> :
> | Use Racket navbar on pkgs. and have navbar link to pkgs.
> |
> | Open question: Where does link to Planet go? Sam's new design has a natural 
> place, but on the old site... it's not clear, community?
> :
>   M .../meta/pkg-index/official/static/index.html |  2 +
>   M .../meta/pkg-index/official/static/style.css  | 49 
> 
>   M pkgs/plt-services/meta/web/all.rkt|  2 +-
>   C pkgs/plt-services/meta/{web/common => pkg-index/official/static}/logo.png 
> (100%)
>   M pkgs/plt-services/meta/web/config.rkt |  1 +
>   M pkgs/plt-services/meta/web/stubs/all.rkt  |  5 +-
>   C pkgs/plt-services/meta/web/stubs/{planet.rkt => packages.rkt} (75%)
>
> ~~
>
> 9012f7b Jay McCarthy  2013-12-20 16:29
> :
> | Adding suggestion about 1.0 package versions
> :
>   M .../racket-doc/pkg/scribblings/getting-started.scrbl | 5 -
>
> =[ Overall Diff ]===
>
> pkgs/plt-services/meta/pkg-index/official/static/index.html
> ~~~
> --- OLD/pkgs/plt-services/meta/pkg-index/official/static/index.html
> +++ NEW/pkgs/plt-services/meta/pkg-index/official/static/index.html
> @@ -10,6 +10,8 @@
>  
>
>
> + cellpadding="0" cellspacing="0" width="100%"> href="http://racket-lang.org/";>( style="font-size: 80px; vertical-align: middle;">( class="navtitle" style="font-size: 60px; vertical-align: 
> middle;">(  style="vertical-align: middle; margin: 13px 0.25em 0 0; border: 0;"> class="navtitle" style="font-size: 80px; vertical-align: 
> middle;">Racket  style="font-size: 60px; vertical-align: middle;">) class="navtitle" style="font-size: 80px; vertical-align: 
> middle;">)) href="http://racket-lang.org/help.html";>Need 
> Help? width="100%"> class="navlink"> href="http://racket-lang.org/";>About class="navlinkcell"> href="http://racket-lang.org/download/";>Download class="navlinkcell"> href="http://docs.racket-lang.org/";>Documentation class="navlinkcell"> href="/">Packages class="navitem"> href="http://racket-lang.org/community.html";>Community  class="navlinkcell"> href="http://racket-lang.org/learning.html!
 ">!
>  Learning!
>  
> +
>  
>Packages
>
> @@ -55,7 +57,7 @@
>  Description: id="pi_description">
>  Tags:
>   id="pi_add_tag_text" class="text ui-widget-content ui-corner-all" /> id="pi_add_tag_button">Add Tag
> -Versions Exceptions id="pi_versions">
> +Version Exceptions id="pi_versions">
>  Version: 
> Source:  id="pi_add_version_source_text" class="text ui-widget-content ui-corner-all" 
> />Add Version Exception
>  Dependencies id="pi_dependencies">
>  Conflicts id="pi_conflicts">
>
> pkgs/plt-services/meta/pkg-index/official/static/style.css
> ~~
> --- OLD/pkgs/plt-services/meta/pkg-index/official/static/style.css
> +++ NEW/pkgs/plt-services/meta/pkg-index/official/static/style.css
> @@ -154,3 +154,52 @@ a.possible {
>  tr#pi_delete_row td {
>  text-align: center;
>  }
> +
> +/* racket nav */
> +.racketnav {
> +  background-color: #00;
> +  color: #ff;
> +  margin-bottom: 1em;
> +  padding: 0.5em 0em;
> +  white-space: nowrap;
> +}
> +.racketnav a {
> +  color: #ff;
> +  text-decoration: none;
> +}
> +.racketnav .navcontent {
> +  margin-left: auto;
> +  margin-right: auto;
> +  width: 45em;
> +  font-family: Optima, Arial, Verdana, Helvetica, sans-serif;
> +}
> +.racketnav .navtitle {
> +  font-size: xx-large;
> +  font-weight: bold;
> +}
> +.racketnav .navitem {
> +  text-decoration: none;
> +  font-size: 88%;
> +}
> +.racketnav .navlink a {
> +  padding: 0em 1em;
> +}
> +.racketnav .navcurlink a {
> +  padding: 0em 1em;
> +  background-color: #55;
> +}
> +.racketnav .navlinka:hover,
> +.racketnav .navcurlink