Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should be easy to create have a byline #741

Closed
djay opened this issue Jul 21, 2015 · 15 comments · Fixed by plone/plone.app.layout#59 or plone/plonetheme.barceloneta#43
Closed

Should be easy to create have a byline #741

djay opened this issue Jul 21, 2015 · 15 comments · Fixed by plone/plone.app.layout#59 or plone/plonetheme.barceloneta#43

Comments

@djay
Copy link
Member

djay commented Jul 21, 2015

User problem

Some sites show some combination of

  • last updated date
  • last published date
  • creators name
  • last editors name

The person in charge of deciding what information is shown on the page (information architect/content team leader/site admin/themer) should be able to include this information in the page easily and flexibly being able to pick which of the above they want to show. Showing at least the publication date is a common feature of blog posts and news items in other CMS's

History

The byline was a feature used both by internal editors and reused to show to the public (with reduced functionality, ie no history). Currently for 5.0 it is hidden by default because there is last updated date is shown in the toolbar and the history view provides the other info at a single click.

replaces #668 and #714

Options

  1. Have a tickbox to turn the byline on
    • Fix the current tickboxes in security and site which previously determine if the byline is visible to the public and if it should show last published date. This would turn it on for everyone or off for everyone.
      • should probably move somewhere more sensible like theming.
      • improve them to give more options about showing date without author Unlink display of the author's display date. #668
      • pro: backwards compatible setting.
      • con: makes it harder for themers to cope with visual elements that can be turn on thus breaking the theme (but we already have that with things like TOC)
  2. Have the viewlet hidden but easily enabled by switching on the viewlet ttw. Document how to use diazo to hide parts you don't want.
  3. Always have the byline turned on. Document how to use diazo to hide parts you don't want or all of it
  4. Document how to use diazo to create your own byline using parameter expression like modified = context/modified and a diazo rule like <insert-after css:theme="#content h1"><p>Last modified {$modified}</p></insert-after>
    ~~- pro: teaches them more flexible approach. code is small and easy to copy
    - con: means that have to modify an out of the box theme.
  5. solve "way to include dynamic or rich content into the text of a page" no out of the box way to include dynamic or rich content into the text of a page #469, ie by mosaic or similar. byline would be a tile. or by htmlfragments (requires coding)
  6. have something like collective.listingviews built into plone which enables admins to add portlets with common metadata without coding which can then be styled using diazo.
@davilima6
Copy link
Member

davilima6 commented Jul 21, 2015

You opened a new issue but forgot to mention original thread (#668) most voted suggestion: byline stays as available CMS feature however hidden and marked as a deprecated viewlet that will need to be replaced when Mosaic comes out. By there we'll have a configurable byline tile that will allows us to delete old site setup settings.

As a bonus we avoid changing twice the recommended way of having a byline in Plone.

I also find important to bring in this issue the research by @hvelarde: despite Barceloneta aiming for minimalism, byline may be considered an expected native CMS feature.

@keul
Copy link
Member

keul commented Jul 21, 2015

Probably it's too late for Plone 5, but I think it's time to make the byline a little more configurable.

We really enjoy the works we did with rt.lastmodifier, where there's a specific permission for every piece of the byline (that's is not related to theme but is a security configuration). The global check "show/hide byline to anonymous" is not enough in lot of use cases.

@davilima6
Copy link
Member

Nice! I hope it's not hard porting those features into Mosaic's byline tile.

@hvelarde
Copy link
Member

hvelarde commented Jul 21, 2015

@djay I really appreciate the work you have done trying to resume the conversation we have the other day, but we need to agree on one basic concept before continue working on this issue: the byline is content, not presentation. so, it is not the "themer" who decides if that content is there, but the publisher of a site.

it has to be easy to change the settings and it has to be done independently of the theme; I don't see the need of having Diazo involved on this.

what @keul showed here probably the way we need to go.

@hvelarde
Copy link
Member

hvelarde commented Sep 8, 2015

@vangheem why did you closed it? we haven't even starting talking about it :-)

@hvelarde hvelarde reopened this Sep 8, 2015
@mauritsvanrees
Copy link
Member

@hvelarde He closed it automatically by merging a pull request that actually fixes it. Seems legit. :-)

@hvelarde
Copy link
Member

hvelarde commented Sep 8, 2015

@mauritsvanrees no, it doesn't fix it; we're talking about different things here

@davilima6
Copy link
Member

davilima6 commented Sep 9, 2015

@hvelarde is right, only the #714 part of this issue has been closed by plone/plone.app.layout#59 - and even then not completely cause now we have bad redundancy with toolbar: if we are keeping history in the latter we must hide the viewlet by default. Also we should file a bug in which the viewlet doesn't respect its visibility setting in @@site-controlpanel (/cc @rodfersou)

After that we still have to discuss how to implement the controls suggested in #668. I think those should go on Mosaic only (option 5) for the sake of avoiding both coredev rewrites and integrators pain in the near future.

@hvelarde hvelarde reopened this Sep 9, 2015
@esteele esteele added this to the Plone 5.0.x milestone Sep 19, 2015
@rodfersou
Copy link
Member

This issue #1037 closes the part to respect @@site-controlpanel

/cc @davilima6 @hvelarde

@hvelarde
Copy link
Member

hvelarde commented May 5, 2016

I removed the "strange to have some aspects of theme determined by a tickbox and others" line from the "Have a tickbox to turn the byline on" option; as we discussed before, metadata is content, not theme.

@fredvd
Copy link
Member

fredvd commented Oct 7, 2016

Now with the latest released Plone 5.0.6 there's a Security setting in the control panel:
"Allow anyone to view 'about' information" which doesn't do anything for the byline for anonymous vistors because by default the byline viewlet is hidden.

@hvelarde
Copy link
Member

hvelarde commented Oct 7, 2016

@fredvd I fixed that for Plone 5.1

@thet
Copy link
Member

thet commented Dec 13, 2016

Closing this, because following the discussion and linked PR/issues it looks like this issue being fixed.
Reopen if not.

@thet thet closed this as completed Dec 13, 2016
@hvelarde
Copy link
Member

it's not, but I'll create a new one later explaining what's missing; then we can close this one.

@hvelarde hvelarde reopened this Dec 13, 2016
@hvelarde
Copy link
Member

superseded by #1908.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment