address review

Signed-off-by: Simon L <szaimen@e.mail.de>
pull/40784/head
Simon L 2023-10-06 16:08:26 +07:00
parent 6439fdf5b6
commit e5b11ced4a
7 changed files with 125 additions and 28 deletions

@ -77,7 +77,7 @@ return [
['name' => 'TwoFactorSettings#update', 'url' => '/settings/api/admin/twofactorauth', 'verb' => 'PUT' , 'root' => ''], ['name' => 'TwoFactorSettings#update', 'url' => '/settings/api/admin/twofactorauth', 'verb' => 'PUT' , 'root' => ''],
['name' => 'AISettings#update', 'url' => '/settings/api/admin/ai', 'verb' => 'PUT' , 'root' => ''], ['name' => 'AISettings#update', 'url' => '/settings/api/admin/ai', 'verb' => 'PUT' , 'root' => ''],
['name' => 'Help#help', 'url' => '/settings/help', 'verb' => 'GET', 'root' => ''], ['name' => 'Help#help', 'url' => '/settings/help/{mode}', 'verb' => 'GET', 'defaults' => ['mode' => ''] , 'root' => ''],
['name' => 'WebAuthn#startRegistration', 'url' => '/settings/api/personal/webauthn/registration', 'verb' => 'GET' , 'root' => ''], ['name' => 'WebAuthn#startRegistration', 'url' => '/settings/api/personal/webauthn/registration', 'verb' => 'GET' , 'root' => ''],
['name' => 'WebAuthn#finishRegistration', 'url' => '/settings/api/personal/webauthn/registration', 'verb' => 'POST' , 'root' => ''], ['name' => 'WebAuthn#finishRegistration', 'url' => '/settings/api/personal/webauthn/registration', 'verb' => 'POST' , 'root' => ''],

@ -1,7 +1,20 @@
.help-includes {
overflow: hidden !important;
}
.help-list__text { .help-list__text {
margin-left: 24px; margin-left: 24px;
} }
.help-iframe {
width: 100%;
height: 100%;
margin: 0;
padding: 0;
border: 0;
overflow: auto;
}
.help-wrapper { .help-wrapper {
display: flex; display: flex;
justify-content: center; justify-content: center;

@ -39,6 +39,7 @@ use OCP\IL10N;
use OCP\INavigationManager; use OCP\INavigationManager;
use OCP\IRequest; use OCP\IRequest;
use OCP\IURLGenerator; use OCP\IURLGenerator;
use OCP\IConfig;
#[IgnoreOpenAPI] #[IgnoreOpenAPI]
class HelpController extends Controller { class HelpController extends Controller {
@ -55,6 +56,9 @@ class HelpController extends Controller {
/** @var string */ /** @var string */
private $userId; private $userId;
/** @var IConfig */
private $config;
public function __construct( public function __construct(
string $appName, string $appName,
IRequest $request, IRequest $request,
@ -62,7 +66,8 @@ class HelpController extends Controller {
IURLGenerator $urlGenerator, IURLGenerator $urlGenerator,
?string $userId, ?string $userId,
IGroupManager $groupManager, IGroupManager $groupManager,
IL10N $l10n IL10N $l10n,
IConfig $config,
) { ) {
parent::__construct($appName, $request); parent::__construct($appName, $request);
$this->navigationManager = $navigationManager; $this->navigationManager = $navigationManager;
@ -70,6 +75,7 @@ class HelpController extends Controller {
$this->userId = $userId; $this->userId = $userId;
$this->groupManager = $groupManager; $this->groupManager = $groupManager;
$this->l10n = $l10n; $this->l10n = $l10n;
$this->config = $config;
} }
/** /**
@ -79,19 +85,40 @@ class HelpController extends Controller {
* @NoAdminRequired * @NoAdminRequired
* @NoSubAdminRequired * @NoSubAdminRequired
*/ */
public function help(): TemplateResponse { public function help(string $mode = 'user'): TemplateResponse {
$this->navigationManager->setActiveEntry('help'); $this->navigationManager->setActiveEntry('help');
$pageTitle = $this->l10n->t('Nextcloud help overview'); $pageTitle = $this->l10n->t('Administrator documentation');
if ($mode !== 'admin') {
$pageTitle = $this->l10n->t('User documentation');
$mode = 'user';
}
$documentationUrl = $this->urlGenerator->getAbsoluteURL(
$this->urlGenerator->linkTo('', 'core/doc/' . $mode . '/index.html')
);
$urlUserDocs = $this->urlGenerator->linkToRoute('settings.Help.help', ['mode' => 'user']);
$urlAdminDocs = $this->urlGenerator->linkToRoute('settings.Help.help', ['mode' => 'admin']);
$urlUserDocs = $this->urlGenerator->linkToDocs('user'); $knowledgebaseEmbedded = $this->config->getSystemValueBool('knowledgebase.embedded', false);
$urlAdminDocs = $this->urlGenerator->linkToDocs('admin'); if (!$knowledgebaseEmbedded) {
$pageTitle = $this->l10n->t('Nextcloud help overview');
$urlUserDocs = $this->urlGenerator->linkToDocs('user');
$urlAdminDocs = $this->urlGenerator->linkToDocs('admin');
}
$response = new TemplateResponse('settings', 'help', [ $response = new TemplateResponse('settings', 'help', [
'admin' => $this->groupManager->isAdmin($this->userId), 'admin' => $this->groupManager->isAdmin($this->userId),
'url' => $documentationUrl,
'urlUserDocs' => $urlUserDocs, 'urlUserDocs' => $urlUserDocs,
'urlAdminDocs' => $urlAdminDocs, 'urlAdminDocs' => $urlAdminDocs,
'mode' => $mode,
'pageTitle' => $pageTitle, 'pageTitle' => $pageTitle,
'knowledgebaseEmbedded' => $knowledgebaseEmbedded,
]); ]);
$policy = new ContentSecurityPolicy();
$policy->addAllowedFrameDomain('\'self\'');
$response->setContentSecurityPolicy($policy);
return $response; return $response;
} }
} }

@ -1,28 +1,75 @@
<?php <?php
\OC_Util::addStyle('settings', 'help'); \OC_Util::addStyle('settings', 'help');
?> ?>
<?php if ($_['knowledgebaseEmbedded'] === true) : ?>
<div id="app-navigation" role="navigation" tabindex="0">
<ul>
<li>
<a class="icon-user <?php if ($_['mode'] === 'user') {
p('active');
} ?>" <?php if ($_['mode'] === 'user') { print_unescaped('aria-current="page"'); } ?>
href="<?php print_unescaped($_['urlUserDocs']); ?>">
<span class="help-list__text">
<?php p($l->t('User documentation')); ?>
</span>
</a>
</li>
<?php if ($_['admin']) { ?>
<li>
<a class="icon-user-admin <?php if ($_['mode'] === 'admin') {
p('active');
} ?>" <?php if ($_['mode'] === 'admin') { print_unescaped('aria-current="page"'); } ?>
href="<?php print_unescaped($_['urlAdminDocs']); ?>">
<span class="help-list__text">
<?php p($l->t('Administrator documentation')); ?>
</span>
</a>
</li>
<?php } ?>
<div id="app-content"> <li>
<div class="help-wrapper"> <a href="https://docs.nextcloud.com" class="icon-category-office" target="_blank" rel="noreferrer noopener">
<div class="help-content"> <span class="help-list__text">
<h2 class="help-content__heading"> <?php p($l->t('Documentation')); ?>
<?php p($l->t('Nextcloud help overview')); ?> </span>
</h2> </a>
<div class="help-content__body"> </li>
<a class="button" target="_blank" rel="noreferrer noopener" <li>
href="<?php print_unescaped($_['urlAdminDocs']); ?>"> <a href="https://help.nextcloud.com" class="icon-comment" target="_blank" rel="noreferrer noopener">
<?php p($l->t('Administration documentation')); ?> <span class="help-list__text">
</a> <?php p($l->t('Forum')); ?>
<a class="button" target="_blank" rel="noreferrer noopener" </span>
href="<?php print_unescaped($_['urlUserDocs']); ?>"> </a>
<?php p($l->t('Account documentation')); ?> </li>
</a> </div>
<a href="https://docs.nextcloud.com" class="button" target="_blank" rel="noreferrer noopener">
<?php p($l->t('General documentation')); ?> <div id="app-content" class="help-includes">
</a> <iframe src="<?php print_unescaped($_['url']); ?>" class="help-iframe" tabindex="0">
<a href="https://help.nextcloud.com" class="button" target="_blank" rel="noreferrer noopener"> </iframe>
<?php p($l->t('Forum')); ?> </div>
</a> <?php else: ?>
<div id="app-content">
<div class="help-wrapper">
<div class="help-content">
<h2 class="help-content__heading">
<?php p($l->t('Nextcloud help overview')); ?>
</h2>
<div class="help-content__body">
<a class="button" target="_blank" rel="noreferrer noopener"
href="<?php print_unescaped($_['urlUserDocs']); ?>">
<?php p($l->t('Account documentation')); ?>
</a>
<a class="button" target="_blank" rel="noreferrer noopener"
href="<?php print_unescaped($_['urlAdminDocs']); ?>">
<?php p($l->t('Administration documentation')); ?>
</a>
<a href="https://docs.nextcloud.com" class="button" target="_blank" rel="noreferrer noopener">
<?php p($l->t('General documentation')); ?>
</a>
<a href="https://help.nextcloud.com" class="button" target="_blank" rel="noreferrer noopener">
<?php p($l->t('Forum')); ?>
</a>
</div>
</div> </div>
</div> </div>
</div> <?php endif; ?>

@ -234,6 +234,12 @@ $CONFIG = [
*/ */
'knowledgebaseenabled' => true, 'knowledgebaseenabled' => true,
/**
* ``true`` embeds the documentation in an iframe inside Nextcloud.
* ``false`` only shows buttons to the online documentation.
*/
'knowledgebase.embedded' => false,
/** /**
* ``true`` allows users to change their display names (on their Personal * ``true`` allows users to change their display names (on their Personal
* pages), and ``false`` prevents them from changing their display names. * pages), and ``false`` prevents them from changing their display names.

@ -0,0 +1,2 @@
Here goes the admin documentation.
In the meantime go to <a href="https://nextcloud.com/support/" target="_blank">nextcloud.com/support/</a>

@ -0,0 +1,2 @@
Here goes the user documentation
In the meantime go to <a href="https://nextcloud.com/support/" target="_blank">nextcloud.com/support/</a>