----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128705/#review98571 -----------------------------------------------------------
doc/glossary/index.docbook (line 44) <https://git.reviewboard.kde.org/r/128705/#comment66386> Maybe we could use something generic, on the line of "based on the core libraries projects by the KDE community, currently KDE Frameworks and previously KDE Platform" doc/glossary/index.docbook (line 137) <https://git.reviewboard.kde.org/r/128705/#comment66388> If Workspace refers to the old name of Plasma, shouldn't it be Workspaces? doc/glossary/index.docbook (line 175) <https://git.reviewboard.kde.org/r/128705/#comment66389> Yes, IIRC "KDE Platform" is the official name of kdelibs4. doc/glossary/index.docbook (line 276) <https://git.reviewboard.kde.org/r/128705/#comment66390> Workspace->Workspaces Maybe I would add a line which explain what it does before the history (this was a problem even before) doc/glossary/index.docbook (line 299) <https://git.reviewboard.kde.org/r/128705/#comment66391> Workspaces doc/glossary/index.docbook (line 324) <https://git.reviewboard.kde.org/r/128705/#comment66392> See above doc/glossary/index.docbook (line 355) <https://git.reviewboard.kde.org/r/128705/#comment66393> From Plasma 5 it's only Plasma. KDE Plasma Workspaces was the name in the Qt4 world. In fact, despite my comments above, we could focus on using only Plasma without the "Workspaces" part. doc/glossary/index.docbook (line 382) <https://git.reviewboard.kde.org/r/128705/#comment66394> Most likely to be removed, yes doc/glossary/index.docbook (line 433) <https://git.reviewboard.kde.org/r/128705/#comment66395> See above for Workspace/Workspaces/none doc/glossary/index.docbook (line 439) <https://git.reviewboard.kde.org/r/128705/#comment66396> See above for Workspace/Workspaces/none doc/glossary/index.docbook (line 444) <https://git.reviewboard.kde.org/r/128705/#comment66397> See above for Workspace/Workspaces/none doc/glossary/index.docbook (line 455) <https://git.reviewboard.kde.org/r/128705/#comment66398> See above for Workspace/Workspaces/none doc/glossary/index.docbook (line 466) <https://git.reviewboard.kde.org/r/128705/#comment66399> See above for Workspace/Workspaces/none doc/glossary/index.docbook (line 471) <https://git.reviewboard.kde.org/r/128705/#comment66400> See above for Workspace/Workspaces/none doc/glossary/index.docbook (line 532) <https://git.reviewboard.kde.org/r/128705/#comment66401> which resources? doc/glossary/index.docbook (line 558) <https://git.reviewboard.kde.org/r/128705/#comment66402> See above for Workspace/Workspaces/none doc/glossary/index.docbook (line 579) <https://git.reviewboard.kde.org/r/128705/#comment66403> strigi does not exist in Frameworks 5 iirc. I would at least comment this out if not remove completely. doc/glossary/kdeprintingglossary.docbook (line 41) <https://git.reviewboard.kde.org/r/128705/#comment66404> I suspect that most of if not the entire document refer to kdeprint, which was not available even in the 4 lifecycle. - Luigi Toscano On Aug. 23, 2016, 11:43 a.m., Burkhard Lück wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128705/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2016, 11:43 a.m.) > > > Review request for Documentation. > > > Repository: khelpcenter > > > Description > ------- > > proofread + update > add comments + questions inline > please help to get this in good shape > > > Diffs > ----- > > doc/glossary/index.docbook 3732d95 > doc/glossary/kdeprintingglossary.docbook f9e03dc > > Diff: https://git.reviewboard.kde.org/r/128705/diff/ > > > Testing > ------- > > passes checkXML5 > > > Thanks, > > Burkhard Lück > >
