Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-21 Thread Paolo Caruccio

Alec,

could you replace the pipe with another character (i.e. !) and see what 
happens?

Il giorno giovedì 21 giugno 2012 06:13:01 UTC+2, Alec Taylor ha scritto:

 Confirmed. 

 That pipe code is definitely there (just under the a class=brand 
 href=#web2pytrade;nbsp;/a line). 

 On Thu, Jun 21, 2012 at 5:00 AM, Paolo Caruccio 
 paolo.carucci...@gmail.com wrote: 
  Alec, 
  
  please double check that in views/layout.html exists this code (note 
 the 
  pipe in separators): 
  
  div id=navbar{{='auth' in globals() and auth.navbar(separators=(' 
 ',' | 
  ','')) or ''}}/div 
  
  You should find it at line 90 in last trunk. 
  
  
  Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha 
 scritto: 
  
  Okay, finally got around to testing it. 
  
  Not getting any pipes. 
  
  The right of my navbar is one long string all of the same colour: 
  Welcome username Logout 


-- 





Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-21 Thread Alec Taylor
Confirmed, problem still exists.

Here is the navbar code directly from the source generated
(post-processing to prettify only):

http://jsfiddle.net/ucYGN/

On Thu, Jun 21, 2012 at 7:58 PM, Paolo Caruccio
paolo.carucci...@gmail.com wrote:

 Alec,

 could you replace the pipe with another character (i.e. !) and see what
 happens?

 Il giorno giovedì 21 giugno 2012 06:13:01 UTC+2, Alec Taylor ha scritto:

 Confirmed.

 That pipe code is definitely there (just under the a class=brand
 href=#web2pytrade;nbsp;/a line).

 On Thu, Jun 21, 2012 at 5:00 AM, Paolo Caruccio
 paolo.carucci...@gmail.com wrote:
  Alec,
 
  please double check that in views/layout.html exists this code (note
  the
  pipe in separators):
 
  div id=navbar{{='auth' in globals() and auth.navbar(separators=('
  ',' |
  ','')) or ''}}/div
 
  You should find it at line 90 in last trunk.
 
 
  Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha
  scritto:
 
  Okay, finally got around to testing it.
 
  Not getting any pipes.
 
  The right of my navbar is one long string all of the same colour:
  Welcome username Logout

 --

-- 





Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-21 Thread Paolo Caruccio
Alec, 

it seems that you aren't using last trunk.

For example #navbar missed classes. In last trunk, indeed, we have:

div class=nav pull-right id=navbar
span class=auth_navbarWelcome paolo 
 a 
href=/welcome/default/user/logout?_next=/welcome/default/indexhttp://127.0.0.1:8000/welcome/default/user/logout?_next=/welcome/default/index
Logout/a | 
 a 
href=/welcome/default/user/profile?_next=/welcome/default/indexhttp://127.0.0.1:8000/welcome/default/user/profile?_next=/welcome/default/index
Profile/a | 
 a href=
/welcome/default/user/change_password?_next=/welcome/default/indexhttp://127.0.0.1:8000/welcome/default/user/change_password?_next=/welcome/default/index
Password/a
   /span
/div

please download last trunk and try again. 
Il giorno giovedì 21 giugno 2012 17:53:55 UTC+2, Alec Taylor ha scritto:

 Confirmed, problem still exists. 

 Here is the navbar code directly from the source generated 
 (post-processing to prettify only): 

 http://jsfiddle.net/ucYGN/ 

 On Thu, Jun 21, 2012 at 7:58 PM, Paolo Caruccio 
 paolo.carucci...@gmail.com wrote: 
  
  Alec, 
  
  could you replace the pipe with another character (i.e. !) and see 
 what 
  happens? 
  
  Il giorno giovedì 21 giugno 2012 06:13:01 UTC+2, Alec Taylor ha scritto: 
  
  Confirmed. 
  
  That pipe code is definitely there (just under the a class=brand 
  href=#web2pytrade;nbsp;/a line). 
  
  On Thu, Jun 21, 2012 at 5:00 AM, Paolo Caruccio 
  paolo.carucci...@gmail.com wrote: 
   Alec, 
   
   please double check that in views/layout.html exists this code 
 (note 
   the 
   pipe in separators): 
   
   div id=navbar{{='auth' in globals() and auth.navbar(separators=(' 
   ',' | 
   ','')) or ''}}/div 
   
   You should find it at line 90 in last trunk. 
   
   
   Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha 
   scritto: 
   
   Okay, finally got around to testing it. 
   
   Not getting any pipes. 
   
   The right of my navbar is one long string all of the same colour: 
   Welcome username Logout 
  
  -- 


-- 





Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-21 Thread Alec Taylor
Reading your code example highlights the problem.

There is no | after Welcome paolo

On Fri, Jun 22, 2012 at 2:32 AM, Paolo Caruccio
paolo.carucci...@gmail.com wrote:
 Alec,

 it seems that you aren't using last trunk.

 For example #navbar missed classes. In last trunk, indeed, we have:

 div class=nav pull-right id=navbar
     span class=auth_navbarWelcome paolo
          a
 href=/welcome/default/user/logout?_next=/welcome/default/indexLogout/a
 |
          a
 href=/welcome/default/user/profile?_next=/welcome/default/indexProfile/a
 |
          a
 href=/welcome/default/user/change_password?_next=/welcome/default/indexPassword/a
    /span
 /div

 please download last trunk and try again.
 Il giorno giovedì 21 giugno 2012 17:53:55 UTC+2, Alec Taylor ha scritto:

 Confirmed, problem still exists.

 Here is the navbar code directly from the source generated
 (post-processing to prettify only):

 http://jsfiddle.net/ucYGN/

 On Thu, Jun 21, 2012 at 7:58 PM, Paolo Caruccio
 paolo.carucci...@gmail.com wrote:
 
  Alec,
 
  could you replace the pipe with another character (i.e. !) and see
  what
  happens?
 
  Il giorno giovedì 21 giugno 2012 06:13:01 UTC+2, Alec Taylor ha scritto:
 
  Confirmed.
 
  That pipe code is definitely there (just under the a class=brand
  href=#web2pytrade;nbsp;/a line).
 
  On Thu, Jun 21, 2012 at 5:00 AM, Paolo Caruccio
  paolo.carucci...@gmail.com wrote:
   Alec,
  
   please double check that in views/layout.html exists this code
   (note
   the
   pipe in separators):
  
   div id=navbar{{='auth' in globals() and auth.navbar(separators=('
   ',' |
   ','')) or ''}}/div
  
   You should find it at line 90 in last trunk.
  
  
   Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha
   scritto:
  
   Okay, finally got around to testing it.
  
   Not getting any pipes.
  
   The right of my navbar is one long string all of the same colour:
   Welcome username Logout
 
  --

 --

-- 





Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-21 Thread Paolo Caruccio
So you could modify #navbar in layout.html

For example:

 div id=navbar
 {{if auth.user_id:}}
 {{=auth.navbar(separators=(', ' , ' | ' , ''))}}
 {{else:}}
 {{='auth' in globals() and auth.navbar(separators=('' , ' | ' , '')) or 
''}}
 {{pass}}
 /div

When the user is logged in, after his/her name there will be a comma + 1 
space

Il giorno giovedì 21 giugno 2012 18:54:25 UTC+2, Alec Taylor ha scritto:

 Reading your code example highlights the problem. 

 There is no | after Welcome paolo 

 On Fri, Jun 22, 2012 at 2:32 AM, Paolo Caruccio 
 paolo.carucci...@gmail.com wrote: 
  Alec, 
  
  it seems that you aren't using last trunk. 
  
  For example #navbar missed classes. In last trunk, indeed, we have: 
  
  div class=nav pull-right id=navbar 
  span class=auth_navbarWelcome paolo 
   a 
  
 href=/welcome/default/user/logout?_next=/welcome/default/indexLogout/a 
  | 
   a 
  
 href=/welcome/default/user/profile?_next=/welcome/default/indexProfile/a 

  | 
   a 
  
 href=/welcome/default/user/change_password?_next=/welcome/default/indexPassword/a
  

 /span 
  /div 
  
  please download last trunk and try again. 
  Il giorno giovedì 21 giugno 2012 17:53:55 UTC+2, Alec Taylor ha scritto: 
  
  Confirmed, problem still exists. 
  
  Here is the navbar code directly from the source generated 
  (post-processing to prettify only): 
  
  http://jsfiddle.net/ucYGN/ 
  
  On Thu, Jun 21, 2012 at 7:58 PM, Paolo Caruccio 
  paolo.carucci...@gmail.com wrote: 
   
   Alec, 
   
   could you replace the pipe with another character (i.e. !) and see 
   what 
   happens? 
   
   Il giorno giovedì 21 giugno 2012 06:13:01 UTC+2, Alec Taylor ha 
 scritto: 
   
   Confirmed. 
   
   That pipe code is definitely there (just under the a class=brand 
   href=#web2pytrade;nbsp;/a line). 
   
   On Thu, Jun 21, 2012 at 5:00 AM, Paolo Caruccio 
   paolo.carucci...@gmail.com wrote: 
Alec, 

please double check that in views/layout.html exists this code 
(note 
the 
pipe in separators): 

div id=navbar{{='auth' in globals() and 
 auth.navbar(separators=(' 
',' | 
','')) or ''}}/div 

You should find it at line 90 in last trunk. 


Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha 
scritto: 

Okay, finally got around to testing it. 

Not getting any pipes. 

The right of my navbar is one long string all of the same colour: 
Welcome username Logout 
   
   -- 
  
  -- 


-- 





Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-21 Thread Alec Taylor
Thanks, I'll make those changes.

But did you think it would also be a good change to commit to trunk?

On Fri, Jun 22, 2012 at 4:26 AM, Paolo Caruccio
paolo.carucci...@gmail.com wrote:
 So you could modify #navbar in layout.html

 For example:

  div id=navbar
  {{if auth.user_id:}}
  {{=auth.navbar(separators=(', ' , ' | ' , ''))}}
  {{else:}}

  {{='auth' in globals() and auth.navbar(separators=('' , ' | ' , '')) or
 ''}}
  {{pass}}
  /div

 When the user is logged in, after his/her name there will be a comma + 1
 space

 Il giorno giovedì 21 giugno 2012 18:54:25 UTC+2, Alec Taylor ha scritto:

 Reading your code example highlights the problem.

 There is no | after Welcome paolo

 On Fri, Jun 22, 2012 at 2:32 AM, Paolo Caruccio
 paolo.carucci...@gmail.com wrote:
  Alec,
 
  it seems that you aren't using last trunk.
 
  For example #navbar missed classes. In last trunk, indeed, we have:
 
  div class=nav pull-right id=navbar
      span class=auth_navbarWelcome paolo
           a
 
  href=/welcome/default/user/logout?_next=/welcome/default/indexLogout/a
  |
           a
 
  href=/welcome/default/user/profile?_next=/welcome/default/indexProfile/a
  |
           a
 
  href=/welcome/default/user/change_password?_next=/welcome/default/indexPassword/a
     /span
  /div
 
  please download last trunk and try again.
  Il giorno giovedì 21 giugno 2012 17:53:55 UTC+2, Alec Taylor ha scritto:
 
  Confirmed, problem still exists.
 
  Here is the navbar code directly from the source generated
  (post-processing to prettify only):
 
  http://jsfiddle.net/ucYGN/
 
  On Thu, Jun 21, 2012 at 7:58 PM, Paolo Caruccio
  paolo.carucci...@gmail.com wrote:
  
   Alec,
  
   could you replace the pipe with another character (i.e. !) and see
   what
   happens?
  
   Il giorno giovedì 21 giugno 2012 06:13:01 UTC+2, Alec Taylor ha
   scritto:
  
   Confirmed.
  
   That pipe code is definitely there (just under the a class=brand
   href=#web2pytrade;nbsp;/a line).
  
   On Thu, Jun 21, 2012 at 5:00 AM, Paolo Caruccio
   paolo.carucci...@gmail.com wrote:
Alec,
   
please double check that in views/layout.html exists this code
(note
the
pipe in separators):
   
div id=navbar{{='auth' in globals() and
auth.navbar(separators=('
',' |
','')) or ''}}/div
   
You should find it at line 90 in last trunk.
   
   
Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha
scritto:
   
Okay, finally got around to testing it.
   
Not getting any pipes.
   
The right of my navbar is one long string all of the same colour:
Welcome username Logout
  
   --
 
  --

 --

-- 





Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-20 Thread Alec Taylor
Okay, finally got around to testing it.

Not getting any pipes.

The right of my navbar is one long string all of the same colour:
Welcome username Logout


Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-20 Thread Paolo Caruccio
Alec,

please double check that in views/layout.html exists this code (note the 
pipe in separators):

div id=navbar{{='auth' in globals() and auth.navbar(separators=(' ',' | 
','')) or ''}}/div 

You should find it at line 90 in last trunk.


Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha scritto:

 Okay, finally got around to testing it. 

 Not getting any pipes. 

 The right of my navbar is one long string all of the same colour: 
 Welcome username Logout 



Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-20 Thread Alec Taylor
Confirmed.

That pipe code is definitely there (just under the a class=brand
href=#web2pytrade;nbsp;/a line).

On Thu, Jun 21, 2012 at 5:00 AM, Paolo Caruccio
paolo.carucci...@gmail.com wrote:
 Alec,

 please double check that in views/layout.html exists this code (note the
 pipe in separators):

 div id=navbar{{='auth' in globals() and auth.navbar(separators=(' ',' |
 ','')) or ''}}/div

 You should find it at line 90 in last trunk.


 Il giorno mercoledì 20 giugno 2012 12:59:56 UTC+2, Alec Taylor ha scritto:

 Okay, finally got around to testing it.

 Not getting any pipes.

 The right of my navbar is one long string all of the same colour:
 Welcome username Logout

-- 





Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-18 Thread Alec Taylor
b 1

Also, make it contrast with the background, e.g.: white text colour

On Mon, Jun 18, 2012 at 6:30 PM, Andrew awillima...@gmail.com wrote:
 Looks Great to me !



 On Monday, June 18, 2012 4:38:09 PM UTC+12, Massimo Di Pierro wrote:

 Check what's in trunk now. Let me know if you think it needs improvement.

 On Sunday, 17 June 2012 15:45:41 UTC-5, Cliff Kachinske wrote:

 I agree that the old look, with dividers, looks better.

 I also think the buttons are easier to find on the page, so I would like
 to optionally use them.

 Would you take a patch with the option?


 On Saturday, June 16, 2012 11:30:04 PM UTC-4, Andrew wrote:

 A  week or two ago a change was made to the nav bar to give the auth
 buttons for Login, Register, etc   a button look.   This was part of the 
 fix
 to the dropdown menus to get them to work with bootstrap.  Another change
 was made to revert back to the old look (but only as an attachment to a
 previous post in this list).

 There are two screen shots, current welcome app, and with the buttons
 reverted to the old look.

 Credit to Paolo for making both changes.  Given that the buttons have
 been in trunk for a while, it is worth getting a consensus to keep it or
 revert it.  So:

 Vote:
 A)     KEEP THE BUTTON LOOK
 B)     REVERT TO THE OLD LOOK

 if B)

 B.1)     Revert to the Pipe separator between the options (as shown)
 B.2)     Don't include the Pipe separator.


 B.1) involves updating a recent patch to layout.html:
 div id=navbar{{='auth' in globals() and auth.navbar(separators=('
 ',' | ','')) or ''}}/div

 A patch for B.1) has already been sent to Massimo (that's my vote :) ) ,
 but let's see what the general feeling is.

 Thanks


Re: [web2py] Re: Users Poll: To keep the button look for the welcome app's auth options, or not ?

2012-06-18 Thread Richard Vézina
+1 option to use them!

Richard

On Sun, Jun 17, 2012 at 4:45 PM, Cliff Kachinske cjk...@gmail.com wrote:

 I agree that the old look, with dividers, *looks* better.

 I also think the buttons are easier to find on the page, so I would like
 to optionally use them.

 Would you take a patch with the option?


 On Saturday, June 16, 2012 11:30:04 PM UTC-4, Andrew wrote:

 A  week or two ago a change was made to the nav bar to give the auth
 buttons for Login, Register, etc   a button look.   This was part of the
 fix to the dropdown menus to get them to work with bootstrap.  Another
 change was made to revert back to the old look (but only as an attachment
 to a previous post in this list).

 There are two screen shots, current welcome app, and with the buttons
 reverted to the old look.

 Credit to Paolo for making both changes.  Given that the buttons have
 been in trunk for a while, it is worth getting a consensus to keep it or
 revert it.  So:

 *Vote:*
 *A) KEEP THE BUTTON LOOK*
 *B) REVERT TO THE OLD LOOK*
 *
 *
 *if B) *
 *
 *
 *B.1) Revert to the Pipe separator between the options (as shown)*
 *B.2) Don't include the Pipe separator.*


 B.1) involves updating a recent patch to layout.html:
 div id=navbar{{='auth' in globals() and auth.navbar(separators=(' ','
 | ','')) or ''}}/div

 A patch for B.1) has already been sent to Massimo (that's my vote :) ) ,
 but let's see what the general feeling is.

 Thanks