From 31648d65c8c5c3a54fc36b83871dcc09a9ac982f Mon Sep 17 00:00:00 2001 From: friendica Date: Fri, 2 Jan 2015 19:20:52 -0800 Subject: some work on the edit connection workflow explanation and connedit page. This needs a lot more attention, but it was screaming for any improvement in documenting why we tell you that the permissions have been changed but not saved. --- mod/connedit.php | 20 +++++++++++--------- view/css/mod_connedit.css | 18 +++++++++++++++++- view/js/mod_connedit.js | 2 +- view/tpl/abook_edit.tpl | 32 +++++++++++++++++--------------- 4 files changed, 46 insertions(+), 26 deletions(-) diff --git a/mod/connedit.php b/mod/connedit.php index 5bf9c130b..6729029ac 100644 --- a/mod/connedit.php +++ b/mod/connedit.php @@ -444,32 +444,34 @@ function connedit_content(&$a) { 'url' => $a->get_baseurl(true) . '/network/?f=&cid=' . $contact['abook_id'], 'sel' => '', 'title' => t('View recent posts and comments'), - ), + ) + ); + $buttons = array( array( 'label' => (($contact['abook_flags'] & ABOOK_FLAG_BLOCKED) ? t('Unblock') : t('Block')), 'url' => $a->get_baseurl(true) . '/connedit/' . $contact['abook_id'] . '/block', 'sel' => (($contact['abook_flags'] & ABOOK_FLAG_BLOCKED) ? 'active' : ''), - 'title' => t('Block or Unblock this connection'), + 'title' => t('Block (or Unblock) all communications with this connection'), ), array( 'label' => (($contact['abook_flags'] & ABOOK_FLAG_IGNORED) ? t('Unignore') : t('Ignore')), 'url' => $a->get_baseurl(true) . '/connedit/' . $contact['abook_id'] . '/ignore', 'sel' => (($contact['abook_flags'] & ABOOK_FLAG_IGNORED) ? 'active' : ''), - 'title' => t('Ignore or Unignore this connection'), + 'title' => t('Ignore (or Unignore) all inbound communications from this connection'), ), array( 'label' => (($contact['abook_flags'] & ABOOK_FLAG_ARCHIVED) ? t('Unarchive') : t('Archive')), 'url' => $a->get_baseurl(true) . '/connedit/' . $contact['abook_id'] . '/archive', 'sel' => (($contact['abook_flags'] & ABOOK_FLAG_ARCHIVED) ? 'active' : ''), - 'title' => t('Archive or Unarchive this connection'), + 'title' => t('Archive (or Unarchive) this connection - mark channel dead but keep content'), ), array( 'label' => (($contact['abook_flags'] & ABOOK_FLAG_HIDDEN) ? t('Unhide') : t('Hide')), 'url' => $a->get_baseurl(true) . '/connedit/' . $contact['abook_id'] . '/hide', 'sel' => (($contact['abook_flags'] & ABOOK_FLAG_HIDDEN) ? 'active' : ''), - 'title' => t('Hide or Unhide this connection'), + 'title' => t('Hide or Unhide this connection from your other connections'), ), array( @@ -542,11 +544,15 @@ function connedit_content(&$a) { '$notself' => (($self) ? '' : '1'), '$self' => (($self) ? '1' : ''), '$autolbl' => t('Apply the permissions indicated on this page to all new connections.'), + '$buttons' => (($self) ? '' : $buttons), '$viewprof' => t('View Profile'), '$lbl_slider' => t('Slide to adjust your degree of friendship'), '$slide' => $slide, '$tabs' => $t, '$tab_str' => $tab_str, + '$perms_step1' => t('

Step #1. (Completed).

Create connection with minimal or no permissions.

'), + '$perms_step2' => t('

Step #2. (Incomplete).

Review and/or edit the default individual permissions on this page, if desired.

'), + '$perms_step3' => t('

Step #3. (Incomplete).

Submit this page to apply the selected permissions.

Until this is complete, this connection may have insufficient permission to communicate with you.

'), '$is_pending' => (($contact['abook_flags'] & ABOOK_FLAG_PENDING) ? 1 : ''), '$unapproved' => $unapproved, '$inherited' => t('inherited'), @@ -598,12 +604,8 @@ function connedit_content(&$a) { '$ignored' => (($contact['readonly']) ? t('Currently ignored') : ''), '$archived' => (($contact['archive']) ? t('Currently archived') : ''), '$pending' => (($contact['archive']) ? t('Currently pending') : ''), - '$hidden' => array('hidden', t('Hide this contact from others'), ($contact['hidden'] == 1), t('Replies/likes to your public posts may still be visible')), - '$photo' => $contact['photo'], '$name' => $contact['name'], - '$dir_icon' => $dir_icon, '$alt_text' => $alt_text, - '$sparkle' => $sparkle, '$url' => $url )); diff --git a/view/css/mod_connedit.css b/view/css/mod_connedit.css index e7b93a088..82cc6bc9d 100644 --- a/view/css/mod_connedit.css +++ b/view/css/mod_connedit.css @@ -14,15 +14,31 @@ margin-bottom: 5px !important; } -.abook-pending-contact, .abook-permschange, .abook-autotext { +.abook-pending-contact, .abook-autotext { background: orange; font-weight: bold; margin: 10px; padding: 20px 5px 10px; } +.abook-permschange { + width: 100%; +} + +.abook-perms-steps { + float: left; + width: 200px; + height: 210px; + background: orange; + font-weight: bold; + margin: 10px; + padding: 20px 5px 10px; + +} + .abook-permssave { margin-left: 10px; + clear: both; } #contact-slider { diff --git a/view/js/mod_connedit.js b/view/js/mod_connedit.js index fabf24e95..15b768929 100644 --- a/view/js/mod_connedit.js +++ b/view/js/mod_connedit.js @@ -1,7 +1,7 @@ function abook_perms_msg() { $('.abook-permschange').show(); - $('.abook-permschange').html(aStr['permschange']); +// $('.abook-permschange').html(aStr['permschange']); $('.abook-permssave').show(); } diff --git a/view/tpl/abook_edit.tpl b/view/tpl/abook_edit.tpl index c2c11e4b1..bb20312fa 100755 --- a/view/tpl/abook_edit.tpl +++ b/view/tpl/abook_edit.tpl @@ -7,11 +7,27 @@
{{$tabs}}
+
+{{foreach $buttons as $b }} + +{{/foreach}} {{/if}} -
+
+ + + + + + {{if $last_update}} {{$lastupdtext}} {{$last_update}} @@ -27,7 +43,6 @@ {{/if}} - {{if $self}}
@@ -40,10 +55,6 @@ - - {{if $is_pending}} @@ -63,15 +74,6 @@

{{$permlbl}}

{{$permnote}}
- -
- - - - - -
- -- cgit v1.2.3
{{$them}}{{$me}}