Page MenuHomePhabricator

Links on badge card should be accessible
ClosedPublic

Authored by lpriestley on Apr 13 2016, 4:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 3 2024, 9:43 AM
Unknown Object (File)
Feb 3 2024, 9:43 AM
Unknown Object (File)
Feb 3 2024, 9:43 AM
Unknown Object (File)
Feb 3 2024, 9:43 AM
Unknown Object (File)
Feb 3 2024, 9:43 AM
Unknown Object (File)
Jan 29 2024, 12:11 AM
Unknown Object (File)
Jan 24 2024, 5:20 AM
Unknown Object (File)
Jan 24 2024, 3:11 AM
Subscribers

Details

Summary

Ref T10710

Test Plan

Open user profile with badge, flip badge card, open awarder link.

Diff Detail

Repository
rP Phabricator
Branch
badgecardlink
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningwebroot/rsrc/js/core/behavior-badge-view.js:1JAVELIN5`javelinsymbols` Not In Path
Unit
Tests Passed
Build Status
Buildable 11692
Build 14639: Run Core Tests
Build 14638: arc lint + arc unit

Event Timeline

lpriestley retitled this revision from to Links on badge card should be accessible.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.

Can you update the color of the link as well in CSS? it should be {$darkbluetext}.

lpriestley edited edge metadata.

Correct way to specify the link color for badge view?

epriestley edited edge metadata.
epriestley added inline comments.
src/view/page/PhabricatorStandardPageView.php
251

This should still be toggle-class -- otherwise, you'll break existing toggle-class use sites, by never initializing that behavior.

src/view/phui/PHUIBadgeView.php
70

There needs to be an initBehavior('badge-view') in here somewhere.

This revision is now accepted and ready to land.Apr 13 2016, 6:40 PM
lpriestley marked 2 inline comments as done.
lpriestley edited edge metadata.

Fixing where new behavior is initialized

This revision was automatically updated to reflect the committed changes.