Differential D22240

Add a reference to planet.kde.org to the News section of kde.org
ClosedPublic
Actions

Authored by felixernst on Jul 3 2019, 1:40 PM.

Details

Reviewers
kde-www
ognarb
jriddell
Summary

This patch makes planet.kde.org more discoverable by putting a
reference to it at the end of the "News" section of index.php.
The blog posts should be more discoverable because they generally have
a high quality. This should increase engagement of users with the
community and hopefully lead to more contributors.

This reference should also be visible if the "News" feed is not
available so the "News" section containing the reference is moved
outside the "if (defined("FRONTPAGE_LIVE_FEED"))"-node.

Test Plan

Before:

After:

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
felixernst requested review of this revision.Jul 3 2019, 1:40 PM
felixernst created this revision.
felixernst edited the test plan for this revision. (Show Details)Jul 3 2019, 1:47 PM
felixernst added a project: Websites.
felixernst added a subscriber: Websites.
skadinna added a subscriber: skadinna.Jul 3 2019, 2:06 PM

I'm not a reviewer, but you have a few minor issues:

News from KDE members

  • I'd write "contributors" instead of "members"

Read unofficial blog posts

  • Maybe just call them "community blog posts"?
ognarb requested changes to this revision.Jul 3 2019, 2:08 PM

See @skadinna comment. Otherwise, from a technical point of view it's good. 👍

This revision now requires changes to proceed.Jul 3 2019, 2:08 PM
felixernst updated this revision to Diff 61073.Jul 3 2019, 2:13 PM

Change wording like @skadinna suggested

felixernst added a comment.Jul 3 2019, 2:15 PM

Oops, I'll try that again.

felixernst updated this revision to Diff 61074.Jul 3 2019, 2:17 PM

Change wording like @skadinna suggested - this time for real

ognarb accepted this revision.Jul 3 2019, 2:28 PM

Thanks

@jriddell If you are happy with this change, you can deploy it ;)

This revision is now accepted and ready to land.Jul 3 2019, 2:28 PM
felixernst edited the test plan for this revision. (Show Details)Jul 3 2019, 2:31 PM
felixernst updated this revision to Diff 61082.Jul 3 2019, 3:50 PM

Change "News from KDE contributors" to "News from contributors"

There is no need to specify "KDE" on the front page of KDE.

aacid added a subscriber: aacid.Jul 3 2019, 7:44 PM

I don't think it's really "News" what people write on planet.kde.org, don't know of any better word though

jriddell requested changes to this revision.Jul 4 2019, 1:58 PM

We used to have a broken feature for many years listing blog posts with headlines and avatar icons for blog authors. I don't think there's must point in just putting in a bigger link to Planet KDE but having the blog titles from the RSS feed same as we do for Dot articles would be good. Having poster avatars for bonus points would be great.

This revision now requires changes to proceed.Jul 4 2019, 1:58 PM
ognarb added a comment.Jul 4 2019, 2:16 PM

If the server has access to the SimpleXML extension, it should be easy to parse the RSS feed https://planet.kde.org/rss20.xml and display the last 4-5 entries. @jriddell Having the avatars will be a bit more difficult, because this information is not included in the rss feed and I'm not sure it's a good idea to parse the raw html ;).

@felixernst do you want to try implementing this functionality?

jriddell added a comment.Jul 4 2019, 2:20 PM
In D22240#490845, @ognarb wrote:

If the server has access to the SimpleXML extension, it should be easy to parse the RSS feed https://planet.kde.org/rss20.xml and display the last 4-5 entries. @jriddell Having the avatars will be a bit more difficult, because this information is not included in the rss feed and I'm not sure it's a good idea to parse the raw html ;).

@felixernst do you want to try implementing this functionality?

Yeah avatars are a nice to have but not too important. It probably needs parsing the Planet KDE config file to find. There is RSS funtionality in the website as that is what is used for the News feed from the Dot.

Don't call Planet KDE "news", that's not what it is. Just call it "Planet KDE Blogs".

felixernst added a comment.EditedJul 4 2019, 3:23 PM
In D22240#490845, @ognarb wrote:

@felixernst do you want to try implementing this functionality?

I haven't done even semi-serious web developement in years so I am not necessarily thrilled to create the listing but I really think having a reference to Planet KDE there would be great so if this is the only way this revision will get accepted I will do it. Parsing the RSS feed shouldn't be too hard after all.
I'll argue one more time for just adding the reference without the feed:

In D22240#490814, @jriddell wrote:

I don't think there's must point in just putting in a bigger link to Planet KDE

The only link so far is in the footer which makes it virtually invisible to most users. Furthermore the link in the footer is titled "Planet KDE" which is no description for its contents. So I really think this change is already a big improvement even without a feed.

In D22240#490402, @aacid wrote:

I don't think it's really "News" what people write on planet.kde.org, don't know of any better word though

I mainly agree. I wouldn't know of a better word either though. I want to mention that it is new or recent stuff but the only english words for this seem to be "News" or "Tidings". Maybe "New articles"?

In D22240#490846, @jriddell wrote:

Just call it "Planet KDE Blogs".

I don't really like that because it is unclear what "Planet KDE" is or who wrote the blogs. Maybe "Blog posts from contributors" would be a good middle ground? Or maybe go with "Community blog posts" like in the link text? I actually even dislike using the word "blog" there as it is an "internet term" but I guess the term is already established enough.

ngraham added a subscriber: ngraham.Jul 4 2019, 3:25 PM
In D22240#490864, @felixernst wrote:
In D22240#490402, @aacid wrote:

I don't think it's really "News" what people write on planet.kde.org, don't know of any better word though

I mainly agree. I wouldn't know of a better word either though. I want to mention that it is new or recent stuff but the only english words for this seem to be "News" or "Tidings". Maybe "New articles"?

Just call it "Planet KDE Blogs".

I don't really like that because it is unclear what "Planet KDE" is or who wrote the blogs. Maybe "Blog posts from contributors" would be a good middle ground? Or maybe go with "Community blog posts" like in the link text? I actually even dislike using the word "blog" there as it is an "internet term" but I guess the term is already established enough.

+1, "Planet KDE" is a meaningless term for the people we want to be looking at this. "Community blog posts" is wordy, but more descriptive and better IMO.

ognarb added a comment.Jul 8 2019, 1:46 PM

I would suggest, that we deploy this patch with "Community blog posts" and when I have a bit more time I will implement the RSS feed feature.

felixernst updated this revision to Diff 61351.Jul 8 2019, 4:24 PM

Add RSS feed with recent blog posts and change wording to "Community blog posts"

felixernst edited the test plan for this revision. (Show Details)Jul 8 2019, 4:26 PM
felixernst added a comment.EditedJul 8 2019, 4:33 PM
In D22240#492315, @ognarb wrote:

and when I have a bit more time I will implement the RSS feed feature.

I didn't want to be a burden. ^^
It worked fairly well the way you suggested.

ognarb accepted this revision.Jul 8 2019, 6:46 PM

Thanks :)

index.php
184

Please use bootstrap m-0, p-0 and list-unstyled classes instead. This should give you the same result.

felixernst updated this revision to Diff 61401.Jul 9 2019, 12:03 PM

Use classes "list-unstyled m-0 p-0"

jriddell accepted this revision.Jul 9 2019, 3:27 PM

This has been committed by Carl along with another commit.

I put it in title case and added a link to planet kde under it

This revision is now accepted and ready to land.Jul 9 2019, 3:27 PM
ognarb closed this revision.Jul 10 2019, 7:30 AM

Revision Contents
Changeset List

PathPackages
Mindex.php (29 lines)

Diff 61351

View Options

index.php

Loading...