Skip to content
Snippets Groups Projects
Commit cc54019c authored by David Siegfried's avatar David Siegfried
Browse files

fix icons und html-structure, closes #1036

Closes #1036

Merge request studip/studip!608
parent 3ccc5474
No related branches found
No related tags found
No related merge requests found
...@@ -26,13 +26,12 @@ ...@@ -26,13 +26,12 @@
</a> </a>
<?php endif ?> <?php endif ?>
</h1> </h1>
<nav>
<?php if ($is_autor && !$is_tutor && $group->id != 'nogroup' && $group->isMember($GLOBALS['user']->id)) : ?> <?php if ($is_autor && !$is_tutor && $group->id != 'nogroup' && $group->isMember($GLOBALS['user']->id)) : ?>
<a href="<?= $controller->link_for('messages/write', [ <a href="<?= $controller->link_for('messages/write', [
'group_id' => $group->id, 'group_id' => $group->id,
'default_subject' => htmlReady($course_title . ' (' . $group->name . ')'), 'default_subject' => htmlReady($course_title . ' (' . $group->name . ')'),
]) ?>" data-dialog="size=auto"> ]) ?>" data-dialog="size=auto">
<?= Icon::create('mail', 'clickable', ['title' => sprintf( <?= Icon::create('mail', Icon::ROLE_CLICKABLE, ['title' => sprintf(
_('Nachricht an alle Mitglieder der Gruppe %s schicken'), _('Nachricht an alle Mitglieder der Gruppe %s schicken'),
$group->name $group->name
), ),
...@@ -40,27 +39,29 @@ ...@@ -40,27 +39,29 @@
</a> </a>
<?php endif ?> <?php endif ?>
<?php if (!$GLOBALS['perm']->have_perm('admin')) : ?> <?php if (!$GLOBALS['perm']->have_perm('admin')) : ?>
<nav>
<?php if ($group->id != 'nogroup' && $joinable) : ?> <?php if ($group->id != 'nogroup' && $joinable) : ?>
<a href="<?= $controller->url_for('course/statusgroups/join', $group->id) ?>"> <a href="<?= $controller->url_for('course/statusgroups/join', $group->id) ?>">
<?= Icon::create('door-enter', 'clickable', <?= Icon::create('door-enter', Icon::ROLE_CLICKABLE,
['title' => sprintf(_('Mitglied von Gruppe %s werden'), ['title' => sprintf(_('Mitglied von Gruppe %s werden'),
htmlReady($group->name))]) ?></a> htmlReady($group->name))]) ?></a>
<?php elseif ($group->id != 'nogroup' && $group->selfassign && <?php elseif ($group->id != 'nogroup' && $group->selfassign &&
$group->selfassign_start > time()) : ?> $group->selfassign_start > time()) : ?>
<?= Icon::create('door-enter', 'inactive', <?= Icon::create('door-enter', Icon::ROLE_INACTIVE,
['title' => sprintf(_('Der Eintrag in diese Gruppe ist möglich ab %s.'), ['title' => sprintf(_('Der Eintrag in diese Gruppe ist möglich ab %s.'),
date('d.m.Y H:i', $group->selfassign_start))]) ?> date('d.m.Y H:i', $group->selfassign_start))]) ?>
<?php elseif ($group->id != 'nogroup' && $group->selfassign && <?php elseif ($group->id != 'nogroup' && $group->selfassign &&
$group->selfassign_end && $group->selfassign_end < time()) : ?> $group->selfassign_end && $group->selfassign_end < time()) : ?>
<?= Icon::create('door-enter', 'inactive', <?= Icon::create('door-enter', Icon::ROLE_INACTIVE,
['title' => sprintf(_('Der Eintrag in diese Gruppe war möglich bis %s.'), ['title' => sprintf(_('Der Eintrag in diese Gruppe war möglich bis %s.'),
date('d.m.Y H:i', $group->selfassign_end))]) ?> date('d.m.Y H:i', $group->selfassign_end))]) ?>
<?php elseif ($group->id != 'nogroup' && $group->userMayLeave($GLOBALS['user']->id)) : ?> <?php elseif ($group->id != 'nogroup' && $group->userMayLeave($GLOBALS['user']->id)) : ?>
<a href="<?= $controller->url_for('course/statusgroups/leave', $group->id) ?>" data-confirm="<?= sprintf(_('Aus Gruppe %s austragen'),htmlReady($group->name)) . '?' ?>"> <a href="<?= $controller->url_for('course/statusgroups/leave', $group->id) ?>" data-confirm="<?= sprintf(_('Aus Gruppe %s austragen'),htmlReady($group->name)) . '?' ?>">
<?= Icon::create('door-leave', 'attention', <?= Icon::create('door-leave', Icon::ROLE_ATTENTION,
['title' => sprintf(_('Aus Gruppe %s austragen'), ['title' => sprintf(_('Aus Gruppe %s austragen'),
htmlReady($group->name))]) ?></a> htmlReady($group->name))]) ?></a>
<?php endif ?> <?php endif ?>
</nav>
<?php endif ?> <?php endif ?>
<?php if ($is_tutor) : ?> <?php if ($is_tutor) : ?>
<?php if ($group->id != 'nogroup') : ?> <?php if ($group->id != 'nogroup') : ?>
...@@ -71,7 +72,7 @@ ...@@ -71,7 +72,7 @@
'default_subject' => htmlReady($course_title . ' (' . $group->name . ')'), 'default_subject' => htmlReady($course_title . ' (' . $group->name . ')'),
]), ]),
_('Nachricht schicken'), _('Nachricht schicken'),
Icon::create('mail', 'clickable', [ Icon::create('mail', Icon::ROLE_CLICKABLE, [
'title' => sprintf( 'title' => sprintf(
_('Nachricht an alle Mitglieder der Gruppe %s schicken'), _('Nachricht an alle Mitglieder der Gruppe %s schicken'),
$group->name $group->name
...@@ -87,8 +88,7 @@ ...@@ -87,8 +88,7 @@
->setSearchObject($memberSearch) ->setSearchObject($memberSearch)
->setDefaultSelectedUser($group->members->pluck('user_id')) ->setDefaultSelectedUser($group->members->pluck('user_id'))
->setDataDialogStatus(Request::isXhr()) ->setDataDialogStatus(Request::isXhr())
->setJSFunctionOnSubmit(Request::isXhr() ? ->setJSFunctionOnSubmit(Request::isXhr() ? 'STUDIP.Dialog.close();' : false)
'STUDIP.Dialog.close();' : false)
->setExecuteURL($controller->url_for('course/statusgroups/add_member/' . ->setExecuteURL($controller->url_for('course/statusgroups/add_member/' .
$group->id)) $group->id))
->addQuickfilter(_('Veranstaltungsteilnehmende'), ->addQuickfilter(_('Veranstaltungsteilnehmende'),
...@@ -104,16 +104,14 @@ ...@@ -104,16 +104,14 @@
->setSearchObject($memberSearch) ->setSearchObject($memberSearch)
->setDefaultSelectedUser($group->members->pluck('user_id')) ->setDefaultSelectedUser($group->members->pluck('user_id'))
->setDataDialogStatus(Request::isXhr()) ->setDataDialogStatus(Request::isXhr())
->setJSFunctionOnSubmit(Request::isXhr() ? ->setJSFunctionOnSubmit(Request::isXhr() ? 'STUDIP.Dialog.close();' : false)
'STUDIP.Dialog.close();' : false) ->setExecuteURL($controller->url_for('course/statusgroups/add_member/' . $group->id))
->setExecuteURL($controller->url_for('course/statusgroups/add_member/' .
$group->id))
) )
->conditionAll(!$is_locked) ->conditionAll(!$is_locked)
->addLink( ->addLink(
$controller->url_for('course/statusgroups/edit', $group->id), $controller->url_for('course/statusgroups/edit', $group->id),
_('Bearbeiten'), _('Bearbeiten'),
Icon::create('edit', 'clickable', [ Icon::create('edit', Icon::ROLE_CLICKABLE, [
'title' => sprintf( 'title' => sprintf(
_('Gruppe %s bearbeiten'), _('Gruppe %s bearbeiten'),
$group->name $group->name
...@@ -124,7 +122,7 @@ ...@@ -124,7 +122,7 @@
->addLink( ->addLink(
$controller->url_for('course/statusgroups/delete', $group->id), $controller->url_for('course/statusgroups/delete', $group->id),
_('Löschen'), _('Löschen'),
Icon::create('trash', 'clickable', [ Icon::create('trash', Icon::ROLE_CLICKABLE, [
'title' => sprintf( 'title' => sprintf(
_('Gruppe %s löschen'), _('Gruppe %s löschen'),
$group->name $group->name
...@@ -133,17 +131,17 @@ ...@@ -133,17 +131,17 @@
['data-confirm' => _('Soll die Gruppe wirklich gelöscht werden?')] ['data-confirm' => _('Soll die Gruppe wirklich gelöscht werden?')]
) ?> ) ?>
<?php else : ?> <?php else : ?>
<nav>
<a href="<?= $controller->url_for('messages/write', [ <a href="<?= $controller->url_for('messages/write', [
'filter' => 'not_grouped', 'filter' => 'not_grouped',
'course_id' => $course_id, 'course_id' => $course_id,
'default_subject' => htmlReady($course_title).' ('.htmlReady($group->name).')' 'default_subject' => htmlReady($course_title).' ('.htmlReady($group->name).')'
]) ?>" data-dialog="size=auto;"> ]) ?>" data-dialog="size=auto;">
<?= Icon::create('mail', 'clickable', [ <?= Icon::create('mail')->asImg(['title' => _('Nachricht an alle nicht zugeordneten Personen schicken')]) ?>
'title' => _('Nachricht an alle nicht zugeordneten Personen schicken') </a>
])->asImg(20) ?></a> </nav>
<?php endif ?> <?php endif ?>
<?php endif ?> <?php endif ?>
</nav>
</header> </header>
<section> <section>
<article id="group-members-<?= $group->id ?>"<?= $load ? ' class="open"' : ''?>> <article id="group-members-<?= $group->id ?>"<?= $load ? ' class="open"' : ''?>>
......
...@@ -150,12 +150,12 @@ section.contentbox { ...@@ -150,12 +150,12 @@ section.contentbox {
align-items: center; align-items: center;
} }
nav { nav {
a, span { a, > span {
display: inline-block; display: inline-block;
vertical-align: middle; vertical-align: middle;
} }
span { > span {
&:last-child { &:last-child {
border-right: none; border-right: none;
padding-right: 0px; padding-right: 0px;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment