[PLIP-Advisories] Re: [Plone] #9288: Improved commenting infrastructure
#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
#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
#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
#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
#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
#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
#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
#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
#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
#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
#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
#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
#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
#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
#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.
#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
#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
#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
#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
#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
#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
#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
#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
#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