Differential D14018

Break long strings in all remarkup areas
ClosedPublic
Actions

Authored by chad on Aug 31 2015, 7:59 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll

Commits

Restricted Diffusion Commit
rP42ed5241204d: Break long strings in all remarkup areas

Summary

Fixes T9264. I'm surprised this hasn't come up before, but any long string or URL in remarkup will overflow and cause remarkup areas to scroll. Prefer breaking these words.

Test Plan

Review a timeline feed in Differential and a Ponder answer.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 33900.Aug 31 2015, 7:59 PM

chad retitled this revision from to Break long strings in all remarkup areas.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

chad added a task: T9264: Ponder Question/Answer on Mobile display is not able to horizontally scroll.

Herald added a subscriber: Korvin. · View Herald TranscriptAug 31 2015, 7:59 PM

chad edited the test plan for this revision. (Show Details)Aug 31 2015, 8:00 PM

should be break-word

chad edited the test plan for this revision. (Show Details)Aug 31 2015, 8:02 PM

epriestley accepted this revision.Sep 3 2015, 2:15 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Sep 3 2015, 2:15 PM

Closed by commit rP42ed5241204d: Break long strings in all remarkup areas (authored by chad, committed by chad). · Explain WhySep 3 2015, 6:03 PM

This revision was automatically updated to reflect the committed changes.

Pawka mentioned this in T9522: CSS rule "word-break: break-word" breaks line numbers on innline pastes and table headers on phriction documents.Oct 7 2015, 7:33 AM

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

core/

remarkup.css

1 line

Diff 33976

View Options

resources/celerity/map.php