[PLIP-Advisories] Re: [Plone] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Comment(by esteele): (In [30486]) Merge in #9286, refs #9286. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#comment:36 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] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: closed Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: fixed Keywords: portlets| +--- Changes (by esteele): * status: assigned = closed * resolution: = fixed Comment: (In [30487]) Remove PLIP branches. Closes #9286. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#comment:37 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] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Comment(by esteele): This PLIP has been accepted for merging into Plone 4.0 The final vote was: Alec Mitchell +1 David Glick +1 Erik Rose +1 Laurence Rowe +1 Matthew Wilkes - Ross Patterson +1 Please merge your branches into the Plone 4.0 head by end-of-day Friday Oct 16. If you need assistance with merging, please contact me. We'll be assigning a documentation ticket to this PLIP shortly. Please assist the docs team in documenting the changes and new features that this PLIP introduces. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#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] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Comment(by esteele): Please assist the doc team in creating/updating documentation relating to this PLIP. See #9286. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#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
[PLIP-Advisories] Re: [Plone] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Comment(by esteele): Your PLIP has been reviewed by the Framework team. Feel free to discuss any suggested changes either here in the PLIP ticket or on the mailing lists. Final deadline for this PLIP is set for September 23. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#comment:26 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] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Comment(by erikrose): (In [29510]) Completed my review of the show/hide portlets PLIP. Refs #9286. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#comment:25 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] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Old description: '''Proposer:''' Ricardo Alves[[BR]] '''Seconder:''' [[BR]] == Motivation == Currently, only the static text portlet can be hidden. This option should be available to all portlets in a more UI friendly away. == Assumptions == The need to hide/disable portlets is very common. Most portlets have persistent settings, so the ability to keep them configured but hidden can make a difference. The portlets themselves may implement this feature individually (as the static portlet does), but there should be a more generic solution, allowing to show/hide portlets at the management screen. == Proposal == Allow the user to hide individual portlets using the portlet management interface with a single click. Hidden portlets should be easily identified, e.g. using a different background color (gray), suggesting that the portlet is disabled. == Implementation == This can be done using a simple, clickable show/hide icon, and some AJAX to perform the server changes. The show/hide option should be stored as an annotation (not at the portlet schema as currently implemented by the static portlet). The hide field should be removed from the static text portlet schema. The GenericSetup import/export handler at plone.app.portlets should be updated, allowing to set the hidden flag to portlet assignments via GenericSetup. In regarding to backwards compatibility, it should be fairly easy to keep old static text portlets working, based on the old value stored or, if necessary, using a specific migration. == Deliverables == A new release of plone.app.portlets including these features. A new release of plone.portlet.static which removes the hide option. Updated documentation about the static portlet and the portlet management screen to explain the new features. == Risks == The problem of users expecting to find the hide field when editing static text portlets, which is negligible considering the gains in usability. == Participants == - Ricardo Alves (igbun) == Progress == Not implemented yet. New description: '''Proposer:''' Ricardo Alves[[BR]] '''Seconder:''' [[BR]] == Motivation == Currently, only the static text portlet can be hidden. This option should be available to all portlets in a more UI friendly away. == Assumptions == The need to hide/disable portlets is very common. Most portlets have persistent settings, so the ability to keep them configured but hidden can make a difference. The portlets themselves may implement this feature individually (as the static portlet does), but there should be a more generic solution, allowing to show/hide portlets at the management screen. == Proposal == Allow the user to hide individual portlets using the portlet management interface with a single click. Hidden portlets should be easily identified, e.g. using a different background color (gray), suggesting that the portlet is disabled. == Implementation == This can be done using a simple, clickable show/hide icon, and some AJAX to perform the server changes. The show/hide option should be stored as an annotation (not at the portlet schema as currently implemented by the static portlet). The hide field should be removed from the static text portlet schema. The GenericSetup import/export handler at plone.app.portlets should be updated, allowing to set the hidden flag to portlet assignments via GenericSetup. In regarding to backwards compatibility, it should be fairly easy to keep old static text portlets working, based on the old value stored or, if necessary, using a specific migration. == Deliverables == A new release of plone.app.portlets including these features. A new release of plone.portlet.static which removes the hide option. Updated documentation about the static portlet and the portlet management screen to explain the new features. == Risks == The problem of users expecting to find the hide field when editing static text portlets, which is negligible considering the gains in usability. == Participants == - Ricardo Alves (igbun) == Progress == Not implemented yet. == Branches == * source:/plone.app.portlets/branches/plip9286-allow-show-hide-portlets * source:/plone.portlet.static/branches/plip9286-allow-show-hide-portlets -- Comment(by erikrose): Linked to branches. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#comment:24 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories
[PLIP-Advisories] Re: [Plone] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Comment(by igbun): (In [29097]) Adding plip 9286 buildout configuration and description documento. This PLIP is ready for review (refs #9286). -- Ticket URL: http://dev.plone.org/plone/ticket/9286#comment:23 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] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: assigned Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Changes (by grahamperrin): * cc: grahamper...@gmail.com (added) -- Ticket URL: http://dev.plone.org/plone/ticket/9286#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] #9286: Allow to show/hide portlets
#9286: Allow to show/hide portlets +--- Reporter: igbun |Owner: igbun Type: PLIP| Status: new Priority: minor |Milestone: 4.0 Component: Infrastructure | Resolution: Keywords: portlets| +--- Comment(by esteele): Approved by FWT vote. -- Ticket URL: http://dev.plone.org/plone/ticket/9286#comment:18 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