Page MenuHomePhabricator
Differential D13076

Increase CSS specificity on Inline Comment Summary
ClosedPublic
Actions

Authored by chad on May 30 2015, 7:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 10, 6:00 PM
Unknown Object (File)
Tue, Dec 3, 11:37 PM
Unknown Object (File)
Wed, Nov 27, 7:15 PM
Unknown Object (File)
Wed, Nov 27, 3:40 AM
Unknown Object (File)
Thu, Nov 21, 1:18 PM
Unknown Object (File)
Tue, Nov 19, 8:47 PM
Unknown Object (File)
Nov 16 2024, 5:38 AM
Unknown Object (File)
Nov 14 2024, 7:15 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T8354: Inline table cell text doesn't wrap in diff's timeline view
Commits
Restricted Diffusion Commit
rP9708f68e585a: Increase CSS specificity on Inline Comment Summary
Summary

Fixes T8354. Adds a class to each th and td for specificity.

Test Plan

Write a large table as an inline comment, see it wraps property on smaller screens.

Diff Detail

Repository
rP Phabricator
Branch
table-dolumn
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6373
Build 6395: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 31543.May 30 2015, 7:56 PM
chad retitled this revision from to Increase CSS specificity on Inline Comment Summary.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added a task: T8354: Inline table cell text doesn't wrap in diff's timeline view.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 30 2015, 7:56 PM
epriestley accepted this revision.May 31 2015, 1:43 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 31 2015, 1:43 PM
Closed by commit rP9708f68e585a: Increase CSS specificity on Inline Comment Summary (authored by chad, committed by chad). · Explain WhyMay 31 2015, 3:37 PM
This revision was automatically updated to reflect the committed changes.
epriestley added a comment.May 16 2017, 4:55 PM

Were these supposed to be "column", not "dolumn"?

chad added a comment.May 16 2017, 4:56 PM

no.

th = table header
tr = table row
td = table dolumn

epriestley added a comment.May 16 2017, 4:56 PM

oh of course silly me

chad added a comment.May 16 2017, 4:56 PM

im ded

epriestley added a comment.May 16 2017, 4:57 PM

sorry it took me 2 years to get the joke

chad added a comment.May 16 2017, 4:58 PM

it is nice to know i used to have a sense of humor!

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
infrastructure/
diff/
view/
11 lines
webroot/
rsrc/
css/
application/
diff/
4 lines

Diff 31543

View Options

resources/celerity/map.php

Loading...
View Options

src/infrastructure/diff/view/PhabricatorInlineSummaryView.php

Loading...
View Options

webroot/rsrc/css/application/diff/inline-comment-summary.css

Loading...
Phabricator · Built by Phacility