Page MenuHomePhabricator

Some formatting changes for showing auth provider config guidance
ClosedPublic

Authored by amckinley on Apr 11 2019, 8:16 PM.
Tags
None
Referenced Files
F12843978: D20400.id48752.diff
Thu, Mar 28, 11:12 PM
F12838352: D20400.id.diff
Thu, Mar 28, 6:08 PM
Unknown Object (File)
Wed, Mar 27, 1:55 AM
Unknown Object (File)
Wed, Mar 27, 1:55 AM
Unknown Object (File)
Mon, Mar 18, 12:21 AM
Unknown Object (File)
Sun, Mar 10, 1:08 PM
Unknown Object (File)
Feb 7 2024, 5:00 PM
Unknown Object (File)
Feb 3 2024, 10:05 PM
Subscribers

Details

Summary

Ref T7667. On the road to locking the auth config, also clean up some minor UI issues:

  • Only show the warning about not Phacility instance auth if the user isn't a manager (see next diff).
  • When rendering more than one warning in the guidance, add bullets.
  • I didn't like the text in the auth.config-lock config setting.
Test Plan

Loaded the page, saw more reasonable-looking guidance:

Screen Shot 2019-04-11 at 1.04.53 PM.png (579×1 px, 66 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

webroot/rsrc/css/phui/phui-info-view.css
77–79

designer.jpg (900×900 px, 48 KB)

This probably adds bullets all over the place, i.e. not only if there are multiple items. Try creating a task with no title, for example?

src/applications/config/option/PhabricatorAuthenticationConfigOptions.php
88–94

Missing space after "to".

This revision is now accepted and ready to land.Apr 11 2019, 8:20 PM

Oh whoops, I was editing the PhabricatorAuthProvidersGuidanceEngineExtension in rSERVICES.

src/applications/auth/guidance/PhabricatorAuthProvidersGuidanceEngineExtension.php
107

This should not be translated: el-autho.locky-configa is not a valid setting.

107

(The key won't actually translate but %s'ing the value is pointless.)

This probably adds bullets all over the place, i.e. not only if there are multiple items. Try creating a task with no title, for example?

PHUIInfoView does this, which seems to be working:

if (count($errors) > 1) {
  $list = array();
  foreach ($errors as $error) {
    $list[] = phutil_tag(
      'li',
      array(),
      $error);
  }
  $list = phutil_tag(
    'ul',
    array(
      'class' => 'phui-info-view-list',
    ),
    $list);
} else if (count($errors) == 1) {
  $list = head($this->errors);
} else {
  $list = null;
}

I tried making the guidance show a single message and that looked right. Making a task with an empty title doesn't show the bullet either.

PHUIInfoView does this, which seems to be working:

Oh, neat. I had no idea!

This revision was automatically updated to reflect the committed changes.