Page MenuHomePhabricator

Remove server side limit on policy control names
ClosedPublic

Authored by chad on Mar 7 2017, 9:29 PM.
Tags
None
Referenced Files
F13060724: D17479.diff
Fri, Apr 19, 6:18 PM
Unknown Object (File)
Wed, Apr 17, 2:50 PM
Unknown Object (File)
Thu, Apr 11, 7:06 AM
Unknown Object (File)
Tue, Apr 9, 5:19 AM
Unknown Object (File)
Sun, Apr 7, 6:52 PM
Unknown Object (File)
Sat, Apr 6, 7:33 AM
Unknown Object (File)
Thu, Apr 4, 11:51 AM
Unknown Object (File)
Mon, Apr 1, 4:42 PM
Subscribers

Details

Summary

Fixes T12367. CSS here already truncates (or should have been) and is generally more effective. Remove the unneeded server side truncation. Any other UI place these render?

Test Plan

Set Policy to a group name of "Stanford University: Alumni Association and Friends" and see better truncation.

Diff Detail

Repository
rP Phabricator
Branch
policy-control-2 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15936
Build 21107: Run Core Tests
Build 21106: arc lint + arc unit

Event Timeline

eliaspro added a subscriber: eliaspro.

Looks good here! Thanks!

This revision is now accepted and ready to land.Mar 7 2017, 9:34 PM
This revision was automatically updated to reflect the committed changes.