diff options
author | Lukas Fleischer <archlinux@cryptocrack.de> | 2013-08-27 02:07:50 +0200 |
---|---|---|
committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2013-08-27 02:10:13 +0200 |
commit | 3bc951e3d87eaf692a7e47cf16a28d838c7cb2bd (patch) | |
tree | 14a13b7f006b23fd034630a1694456a456b7859e /web | |
parent | be80aa0f01376897c0d7e1533308523d689ea677 (diff) |
pkg_details: Convert most action links to forms
Use forms and POST instead of GET for following actions:
* Flagging/Unflagging a package out-of-date
* Voting for a package and removing votes
* Enabling/Disabling notifications
Use CSS to make the submit buttons of these forms look like links.
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
Diffstat (limited to 'web')
-rw-r--r-- | web/html/css/aur.css | 19 | ||||
-rw-r--r-- | web/template/pkg_details.php | 36 |
2 files changed, 49 insertions, 6 deletions
diff --git a/web/html/css/aur.css b/web/html/css/aur.css index bae8e21..a8fb009 100644 --- a/web/html/css/aur.css +++ b/web/html/css/aur.css @@ -14,3 +14,22 @@ #pkg-stats td.stat-desc { white-space: normal; } + +#actionlist form { + margin: 0; + padding: 0; +} + +#actionlist .text-button { + color: #07b; + background: none; + border: none; + padding: 0; + cursor: pointer; + font-size: 100%; +} + +#actionlist .text-button:hover { + text-decoration: underline; + color: #666; +} diff --git a/web/template/pkg_details.php b/web/template/pkg_details.php index 0973448..bd54923 100644 --- a/web/template/pkg_details.php +++ b/web/template/pkg_details.php @@ -39,20 +39,44 @@ $sources = package_sources($row["ID"]); <li><span class="flagged"><?php if ($row["OutOfDateTS"] !== NULL) { echo __('Flagged out-of-date')." (${out_of_date_time})"; } ?></span></li> <?php if ($USE_VIRTUAL_URLS && $uid): ?> <?php if ($row["OutOfDateTS"] === NULL): ?> - <li><a href="<?= get_pkg_uri($row['Name']) . 'flag/'; ?>"><?= __('Flag package out-of-date'); ?></a></li> + <li> + <form action="<?= get_pkg_uri($row['Name']) . 'flag/'; ?>" method="post"> + <input type="submit" class="button text-button" name="do_Flag" value="<?= __('Flag package out-of-date') ?>" /> + </form> + </li> <?php elseif (($row["OutOfDateTS"] !== NULL) && ($uid == $row["MaintainerUID"] || $atype == "Trusted User" || $atype == "Developer")): ?> - <li><a href="<?= get_pkg_uri($row['Name']) . 'unflag/'; ?>"><?= __('Unflag package'); ?></a></li> + <li> + <form action="<?= get_pkg_uri($row['Name']) . 'unflag/'; ?>" method="post"> + <input type="submit" class="button text-button" name="do_UnFlag" value="<?= __('Unflag package') ?>" /> + </form> + </li> <?php endif; ?> <?php if (user_voted($uid, $row['ID'])): ?> - <li><a href="<?= get_pkg_uri($row['Name']) . 'unvote/'; ?>"><?= __('Remove vote'); ?></a></li> + <li> + <form action="<?= get_pkg_uri($row['Name']) . 'unvote/'; ?>" method="post"> + <input type="submit" class="button text-button" name="do_UnVote" value="<?= __('Remove vote') ?>" /> + </form> + </li> <?php else: ?> - <li><a href="<?= get_pkg_uri($row['Name']) . 'vote/'; ?>"><?= __('Vote for this package'); ?></a></li> + <li> + <form action="<?= get_pkg_uri($row['Name']) . 'vote/'; ?>" method="post"> + <input type="submit" class="button text-button" name="do_Vote" value="<?= __('Vote for this package') ?>" /> + </form> + </li> <?php endif; ?> <?php if (user_notify($uid, $row['ID'])): ?> - <li><a href="<?= get_pkg_uri($row['Name']) . 'unnotify/'; ?>"><?= __('Disable notifications'); ?></a></li> + <li> + <form action="<?= get_pkg_uri($row['Name']) . 'unnotify/'; ?>" method="post"> + <input type="submit" class="button text-button" name="do_UnNotify" value="<?= __('Disable notifications') ?>" /> + </form> + </li> <?php else: ?> - <li><a href="<?= get_pkg_uri($row['Name']) . 'notify/'; ?>"><?= __('Notify of new comments'); ?></a></li> + <li> + <form action="<?= get_pkg_uri($row['Name']) . 'notify/'; ?>" method="post"> + <input type="submit" class="button text-button" name="do_Notify" value="<?= __('Notify of new comments') ?>" /> + </form> + </li> <?php endif; ?> <?php if ($atype == "Trusted User" || $atype == "Developer"): ?> <li><a href="<?= get_pkg_uri($row['Name']) . 'delete/'; ?>"><?= __('Delete Package'); ?></a></li> |