diff --git a/Changes b/Changes index 22d9335..47df8b1 100644 --- a/Changes +++ b/Changes @@ -1,12 +1,18 @@ Revision history for Perl extension PGXN::Manager 0.20.1 - - Removed the `Capfile` and `eg` directory from the source (and distribution). - Examples for managing PGXN can now be found in the pgxn/pgxn-ops - repository on GitHub. - - Replaced all icons with SVG icons from the icons8/flat-color-icons project. + - Removed the `Capfile` and `eg` directory from the source (and + distribution). Examples for managing PGXN can now be found in the + pgxn/pgxn-ops repository on GitHub. + - Replaced all icons with SVG icons from the icons8/flat-color-icons + project. - Updated the favicons for best support on modern browsers, including - SVG and ICO variants, as well as special cases for iOS and Android devices. + SVG and ICO variants, as well as special cases for iOS and Android + devices. + - Added titles to new account moderation buttons. + - Improved the layout of the new account request userinfo popover so + that it is in front of everything else and does not overlap with the + side nav. 0.20.0 2021-05-12T02:15:41Z - Explicitly loading Email::Address in PGXN::Manager::Controller diff --git a/lib/PGXN/Manager/Locale.pm b/lib/PGXN/Manager/Locale.pm index 9efaf24..19fc2a5 100644 --- a/lib/PGXN/Manager/Locale.pm +++ b/lib/PGXN/Manager/Locale.pm @@ -26,6 +26,8 @@ our %Lexicon = ( '"[_1]" is missing the required [numerate,_2,key] [qlist,_3] under [_4]' => '“[_1]” is missing the required [numerate,_2,key] [qlist,_3] under [_4]', '"[_1]" is an invalid distribution name' => '“[_1]” is not a valid distribution name. Distribution names must be at least two characters and may not contain unprintable or whitespace characters or /, \\, or :.', howto_page_title => 'How to create PostgreSQL extensions and distribute them on PGXN', + 'Approve account for "[_1]"' => 'Approve account for “[_1]“', + 'Reject account for "[_1]"' => 'Approve account for “[_1]“', # multimarkdown doc/howto.md | perl -pe 's/([\[\]])/~$1/g' howto_body => q`

PGXN is the PostgreSQL Extension Network. If you’re a PostgreSQL developer, you’ve no doubt created customizations to make your life simpler. This is diff --git a/lib/PGXN/Manager/Templates.pm b/lib/PGXN/Manager/Templates.pm index 3e3af6c..1060aac 100644 --- a/lib/PGXN/Manager/Templates.pm +++ b/lib/PGXN/Manager/Templates.pm @@ -637,8 +637,8 @@ template moderate => sub { cell { class is 'actions'; for my $spec ( - [accept => 'active' ], - [remove => 'deleted' ], + [accept => 'active', T 'Approve account for "[_1]"', $user->{nickname} ], + [remove => 'deleted', T 'Reject account for "[_1]"', $user->{nickname} ], ) { form { class is $spec->[0]; @@ -654,6 +654,7 @@ template moderate => sub { class is 'button'; type is 'image'; name is 'submit'; + title is $spec->[2]; src is $req->uri_for("/ui/img/$spec->[0].svg") }; }; diff --git a/t/moderate.t b/t/moderate.t index b3f63cc..bc60153 100644 --- a/t/moderate.t +++ b/t/moderate.t @@ -4,7 +4,7 @@ use 5.10.0; use utf8; BEGIN { $ENV{EMAIL_SENDER_TRANSPORT} = 'Test' } -use Test::More tests => 289; +use Test::More tests => 291; #use Test::More 'no_plan'; use Plack::Test; use HTTP::Request::Common; @@ -269,6 +269,11 @@ test_psgi +PGXN::Manager::Router->app => sub { $req->uri_for('/ui/img/accept.svg'), '..................... Source should be accept.svg' ); + $tx->is( + './@title', + $mt->maketext('Approve account for "[_1]"', 'bob'), + '..................... Title should be be to approve user', + ); } ); }); @@ -330,6 +335,11 @@ test_psgi +PGXN::Manager::Router->app => sub { $req->uri_for('/ui/img/remove.svg'), '..................... Source should be remove.svg' ); + $tx->is( + './@title', + $mt->maketext('Reject account for "[_1]"', 'bob'), + '..................... Title should be be to reject user', + ); } ); }); diff --git a/www/ui/js/lib.js b/www/ui/js/lib.js index 37948d8..34ba32c 100644 --- a/www/ui/js/lib.js +++ b/www/ui/js/lib.js @@ -40,7 +40,7 @@ PGXN.init_moderate = function () { var bub = $(this).next(); bub.css({ position:'absolute', - left:$(this).offset().left - 64, + // left:$(this).offset().left - 64, top:$(this).offset().top + 21 }).toggle(); bub.click(function () { $(this).hide() }); @@ -79,7 +79,3 @@ PGXN.init_mirrors = function () { $('.actions .button').click(function (e) { PGXN.ajax_click(e, this, '#mirrorlist') }); }); }; - - - -