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)
Sat, May 4, 6:27 PM
Unknown Object (File)
Thu, May 2, 4:42 PM
Unknown Object (File)
Fri, Apr 26, 7:15 PM
Unknown Object (File)
Thu, Apr 25, 7:36 AM
Unknown Object (File)
Wed, Apr 24, 11:03 PM
Unknown Object (File)
Apr 2 2024, 9:48 PM
Unknown Object (File)
Apr 1 2024, 4:51 PM
Unknown Object (File)
Mar 24 2024, 12:26 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 31567

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