1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-10 08:52:39 +01:00

Improve overflow behavior of Timeline view

Summary:
For text like "MMM", make the right parts of the element scroll.

Also fixed a couple of 1px issues here and there.

Test Plan: Added, viewed UIExamples.

Reviewers: chad, btrahan

Reviewed By: chad

CC: aran

Differential Revision: https://secure.phabricator.com/D4152
This commit is contained in:
epriestley 2012-12-11 14:02:51 -08:00
parent d2a5ee4fa4
commit 4a3d50bcb5
3 changed files with 26 additions and 4 deletions

View file

@ -71,6 +71,18 @@ final class PhabricatorTimelineExample extends PhabricatorUIExample {
->appendChild('This event is green!') ->appendChild('This event is green!')
->setColor(PhabricatorTransactions::COLOR_GREEN); ->setColor(PhabricatorTransactions::COLOR_GREEN);
$events[] = id(new PhabricatorTimelineEventView())
->setUserHandle($handle)
->setTitle(str_repeat('Long Text Title ', 64))
->appendChild(str_repeat('Long Text Body ', 64))
->setColor(PhabricatorTransactions::COLOR_ORANGE);
$events[] = id(new PhabricatorTimelineEventView())
->setUserHandle($handle)
->setTitle(str_repeat('LongTextEventNoSpaces', 1024))
->appendChild(str_repeat('LongTextNoSpaces', 1024))
->setColor(PhabricatorTransactions::COLOR_RED);
$colors = array( $colors = array(
PhabricatorTransactions::COLOR_RED, PhabricatorTransactions::COLOR_RED,
PhabricatorTransactions::COLOR_ORANGE, PhabricatorTransactions::COLOR_ORANGE,

View file

@ -197,7 +197,9 @@ final class PhabricatorTimelineEventView extends AphrontView {
array( array(
'class' => implode(' ', $title_classes), 'class' => implode(' ', $title_classes),
), ),
$icon.$title.$extra); $title.$extra);
$title = $icon.$title;
} }
$wedge = phutil_render_tag( $wedge = phutil_render_tag(

View file

@ -43,7 +43,7 @@
} }
.phabricator-timeline-title { .phabricator-timeline-title {
line-height: 27px; line-height: 28px;
min-height: 28px; min-height: 28px;
position: relative; position: relative;
} }
@ -86,14 +86,16 @@
.phabricator-timeline-major-event .phabricator-timeline-title { .phabricator-timeline-major-event .phabricator-timeline-title {
background: #f7f7f7; background: #f7f7f7;
min-height: 29px;
} }
.phabricator-timeline-title { .phabricator-timeline-title {
padding: 0 5px; padding: 0 5px;
overflow-x: auto;
} }
.phabricator-timeline-title-with-icon { .phabricator-timeline-title-with-icon {
padding-left: 36px; padding-left: 33px;
} }
.phabricator-timeline-major-event .phabricator-timeline-content .phabricator-timeline-major-event .phabricator-timeline-content
@ -101,6 +103,11 @@
padding: 10px 15px; padding: 10px 15px;
} }
.phabricator-timeline-core-content {
overflow-x: auto;
}
.device .phabricator-timeline-event-view { .device .phabricator-timeline-event-view {
min-height: 23px; min-height: 23px;
position: relative; position: relative;
@ -125,8 +132,9 @@
width: 30px; width: 30px;
height: 30px; height: 30px;
background-color: #c0c5d1; background-color: #c0c5d1;
top: -1px; top: 0;
left: -3px; left: -3px;
z-index: 3;
} }
.phabricator-timeline-icon { .phabricator-timeline-icon {