[PLIP-Advisories] Re: [Plone] #9288: Improved commenting infrastructure

2009-10-07 Thread plip-advisories
#9288: Improved commenting infrastructure
+---
 Reporter:  timo|Owner:  timo
 Type:  PLIP|   Status:  assigned
 Priority:  minor   |Milestone:  4.0 
Component:  Infrastructure  |   Resolution:  
 Keywords:  |  
+---
Changes (by timo):

 * cc: t...@zmag.de (added)


-- 
Ticket URL: http://dev.plone.org/plone/ticket/9288#comment:31
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9288: Improved commenting infrastructure

2009-10-07 Thread plip-advisories
#9288: Improved commenting infrastructure
+---
 Reporter:  timo|Owner:  timo
 Type:  PLIP|   Status:  assigned
 Priority:  minor   |Milestone:  4.0 
Component:  Infrastructure  |   Resolution:  
 Keywords:  |  
+---

Comment(by timo):

 (In [30244]) Hide the editable-object border in moderation view. elro
 review fix. refs #9288

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9288#comment:32
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9315: New theme for Plone 4

2009-10-07 Thread plip-advisories
#9315: New theme for Plone 4
---+
 Reporter:  limi   |Owner:  limi
 Type:  PLIP   |   Status:  assigned
 Priority:  n/a|Milestone:  4.0 
Component:  Templates/CSS  |   Resolution:  
 Keywords: |  
---+

Comment(by esteele):

 One bug I've found: the dashboard link no longer works, since that anchor
 now opens a menu instead.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9315#comment:41
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #7822: Make standard file content types use ZODB BLOB support

2009-10-07 Thread plip-advisories
#7822: Make standard file content types use ZODB BLOB support
+---
 Reporter:  limi|Owner:  witsch  
 Type:  PLIP|   Status:  assigned
 Priority:  major   |Milestone:  4.0 
Component:  Infrastructure  |   Resolution:  
 Keywords:  focusarea   |  
+---

Comment(by rossp):

 FWT vote, +1 for merge

-- 
Ticket URL: http://dev.plone.org/plone/ticket/7822#comment:65
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9186: Set Image IDs from Title field

2009-10-07 Thread plip-advisories
#9186: Set Image IDs from Title field
+---
 Reporter:  erikrose|Owner:  erikrose
 Type:  PLIP|   Status:  reopened
 Priority:  minor   |Milestone:  4.0 
Component:  Archetypes  |   Resolution:  
 Keywords:  |  
+---

Comment(by rossp):

 FWT vote: +1 for merge

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9186#comment:49
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9214: support logins using e-mail address instead of user id

2009-10-07 Thread plip-advisories
#9214: support logins using e-mail address instead of user id
---+
 Reporter:  davisagli  |Owner:  maurits 
 Type:  PLIP   |   Status:  assigned
 Priority:  minor  |Milestone:  4.0 
Component:  Unknown|   Resolution:  
 Keywords: |  
---+

Comment(by rossp):

 FWT vote: +1 for merging

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9214#comment:38
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9258: Replace Products.ATReferenceBrowserWidget with archetypes.referencebrowserwidget

2009-10-07 Thread plip-advisories
#9258: Replace Products.ATReferenceBrowserWidget with
archetypes.referencebrowserwidget
+---
 Reporter:  tom_gross   |Owner:  tom_gross
 Type:  PLIP|   Status:  assigned 
 Priority:  minor   |Milestone:  4.0  
Component:  Archetypes  |   Resolution:   
 Keywords:  |  
+---

Comment(by rossp):

 FWT vote: +1 for merging

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9258#comment:22
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9263: GenericSetup syntax for importing Sharing page roles

2009-10-07 Thread plip-advisories
#9263: GenericSetup syntax for importing Sharing page roles
+---
 Reporter:  optilude|Owner:  optilude
 Type:  PLIP|   Status:  assigned
 Priority:  major   |Milestone:  4.0 
Component:  Infrastructure  |   Resolution:  
 Keywords:  |  
+---

Comment(by rossp):

 FWT vote: +1 for merge

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9263#comment:27
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9330: Add ability to choose group when adding new site members

2009-10-07 Thread plip-advisories
#9330: Add ability to choose group when adding new site members
-+--
 Reporter:  aclark   |Owner:  aclark  
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by rossp):

 Everyting works for me, can't reproduce MatthewWilkes's error.  Still +1
 for merge.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9330#comment:39
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9311: Clean up of user related actions UI

2009-10-07 Thread plip-advisories
#9311: Clean up of user related actions UI
-+--
 Reporter:  dokter   |Owner:  dokter  
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by rossp):

 Seems incomplete as is.

 FWT vote: -1 for merging

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9311#comment:20
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9315: New theme for Plone 4

2009-10-07 Thread plip-advisories
#9315: New theme for Plone 4
---+
 Reporter:  limi   |Owner:  limi
 Type:  PLIP   |   Status:  assigned
 Priority:  n/a|Milestone:  4.0 
Component:  Templates/CSS  |   Resolution:  
 Keywords: |  
---+

Comment(by rossp):

 FWT vote: +1 for merging

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9315#comment:42
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9315: New theme for Plone 4

2009-10-07 Thread plip-advisories
#9315: New theme for Plone 4
---+
 Reporter:  limi   |Owner:  limi
 Type:  PLIP   |   Status:  assigned
 Priority:  n/a|Milestone:  4.0 
Component:  Templates/CSS  |   Resolution:  
 Keywords: |  
---+

Comment(by limi):

 Replying to [comment:40 MatthewWilkes]:
  This is missing the DTML variable support that I wanted.

 To collect the feedback from FWT channel:

  * base_properties for three color values really doesn't make any sense
  * We want to avoid using variables in CSS, especially using DTML
  * We'll still ship with a deprecated base_properties to ensure products
 don't break, but it will be marked as non-functional as far as the theme
 goes
  * The plan is to work with Rob (of CSSManager) to add logo customization
 + header HTML + ploneCustom.css support in 4.1
  * base_properties has been marked as deprecated since 2.5 (AFAIK)

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9315#comment:43
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9315: New theme for Plone 4

2009-10-07 Thread plip-advisories
#9315: New theme for Plone 4
---+
 Reporter:  limi   |Owner:  limi
 Type:  PLIP   |   Status:  assigned
 Priority:  n/a|Milestone:  4.0 
Component:  Templates/CSS  |   Resolution:  
 Keywords: |  
---+

Comment(by limi):

 Replying to [comment:41 esteele]:
  One bug I've found: the dashboard link no longer works, since that
 anchor now opens a menu instead.

 Yeah, it's one of the things I need some assistance with. There are a
 couple of items that should be in that menu that aren't currently. Thought
 it would be more appropriate to add those after a merge, though.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9315#comment:44
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9315: New theme for Plone 4

2009-10-07 Thread plip-advisories
#9315: New theme for Plone 4
---+
 Reporter:  limi   |Owner:  limi
 Type:  PLIP   |   Status:  assigned
 Priority:  n/a|Milestone:  4.0 
Component:  Templates/CSS  |   Resolution:  
 Keywords: |  
---+

Comment(by limi):

 Oh, and people have started working on the RTL bits now, which makes me
 happy. :)

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9315#comment:45
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9321: Reimplement the search form with an eye on usability

2009-10-07 Thread plip-advisories
#9321: Reimplement the search form with an eye on usability
-+--
 Reporter:  csenger  |Owner:  csenger 
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by erikrose):

 (In [30250]) Updated review for merge vote day. +1 for me. Refs #9321.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9321#comment:31
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9309: Better search for East Asian (multi-byte) languages.

2009-10-07 Thread plip-advisories
#9309: Better search for East Asian (multi-byte) languages.
-+--
 Reporter:  terapyon |Owner:  terapyon
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:  search splitter CJK  |  
-+--

Comment(by rossp):

 Seems like the issues have been addressed.

 FWT vote: +1 for merge

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9309#comment:46
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9330: Add ability to choose group when adding new site members

2009-10-07 Thread plip-advisories
#9330: Add ability to choose group when adding new site members
-+--
 Reporter:  aclark   |Owner:  aclark  
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by erikrose):

 New error: check, say, the Reviewers box when adding a user, forget to
 fill out a username, submit the form, and the Reviewers box is no longer
 checked.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9330#comment:41
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9310: User registration process more flexible

2009-10-07 Thread plip-advisories
#9310: User registration process more flexible
-+--
 Reporter:  dokter   |Owner:  dokter  
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by rossp):

 With r30254 I've fixed the last of my remaining concerns.  I also removed
 the fieldset box in r30253.

 FWT vote: +1 for merge

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9310#comment:43
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9321: Reimplement the search form with an eye on usability

2009-10-07 Thread plip-advisories
#9321: Reimplement the search form with an eye on usability
-+--
 Reporter:  csenger  |Owner:  csenger 
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by rossp):

 FWT vote: +1 for merge

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9321#comment:32
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9330: Add ability to choose group when adding new site members

2009-10-07 Thread plip-advisories
#9330: Add ability to choose group when adding new site members
-+--
 Reporter:  aclark   |Owner:  aclark  
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by erikrose):

 (In [30255]) Changed a has_key (which is deprecated--use x in y instead)
 plus an if to a get for fewer code paths. No behavior changes. Refs #9330.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9330#comment:43
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9330: Add ability to choose group when adding new site members

2009-10-07 Thread plip-advisories
#9330: Add ability to choose group when adding new site members
-+--
 Reporter:  aclark   |Owner:  aclark  
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by rossp):

 (In [30257]) Add postback support so that selected groups are preserved
 when
 validation errors occur.  Refs #9330 comment 41.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9330#comment:44
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9330: Add ability to choose group when adding new site members

2009-10-07 Thread plip-advisories
#9330: Add ability to choose group when adding new site members
-+--
 Reporter:  aclark   |Owner:  aclark  
 Type:  PLIP |   Status:  assigned
 Priority:  minor|Milestone:  4.0 
Component:  Unknown  |   Resolution:  
 Keywords:   |  
-+--

Comment(by rossp):

 Having just looked at #9310 and #9330 again, I realized that merging these
 two PLIPs will mean mostly re-implementing #9330 since #9310 uses formlib
 and #9330 makes changes to the old form controller template.  Since #9310
 preserves the old form controller template, we could still merge #9330 but
 it would be unused by default.

 If the implementer is up to it, however, it should be possible to add a
 field and widget to the #9310 formlib form.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9330#comment:45
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor

2009-10-07 Thread plip-advisories
#9249: Add TinyMCE as the default visual editor
--+-
 Reporter:  limi  |Owner:  robgietema
 Type:  PLIP  |   Status:  assigned  
 Priority:  minor |Milestone:  4.0   
Component:  Visual Editor (Kupu)  |   Resolution:
 Keywords:|  
--+-

Comment(by erikrose):

 In OmniWeb 5.10.1, Safari 4.0.3, and Firefox 3.5.3, once I add any
 Discreet text, all the text I add afterward also ends up discreet, even if
 I switch to a different style or select `(remove style)`. The only way out
 is editing the raw HTML. This makes it a step backward from Kupu, from a
 user's point of view, imo. Once we fix this (and I suspect it's a bug in
 our use of it, as I wasn't  able to reproduce it an TinyMCE's demo site),
 I'll happily change my vote to +1.

 Reproduction:
  1. Edit the front page, and clear out the rich text field.
  1. Type hello on line 1.
  1. Assign the Discreet style to it.
  1. Press Return, and type heading on line 2.
  1. Select `Heading` from the `Style` menu. The Discreet style doesn't go
 away, but it should. No matter how you thrash about with `(remove style)`
 and such, you can't get rid of the `discreet` class on the heading without
 editing the raw HTML.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:33
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories


[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor

2009-10-07 Thread plip-advisories
#9249: Add TinyMCE as the default visual editor
--+-
 Reporter:  limi  |Owner:  robgietema
 Type:  PLIP  |   Status:  assigned  
 Priority:  minor |Milestone:  4.0   
Component:  Visual Editor (Kupu)  |   Resolution:
 Keywords:|  
--+-

Comment(by esteele):

 Erik,

 For what it's worth, I was able to fix this quickly by changing the
 discreet definition in the configlet to use a span instead of p tag. The
 remove formatting method by default only rips out
 span,b,strong,em,i,font,u, and strike elements (see
 
http://dev.plone.org/collective/browser/Products.TinyMCE/trunk/Products/TinyMCE/skins/tinymce/tiny_mce.js#8787).
 I can see several solutions to this and they're all trivial, IMO.

-- 
Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:34
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories