Page MenuHomePhabricator

Legalpad - style NOTE IMPORTANT WARNING remarkup slightly differently
ClosedPublic

Authored by btrahan on Jan 24 2014, 8:41 PM.
Tags
None
Referenced Files
F18820262: D8060.diff
Wed, Oct 22, 1:40 PM
F18783983: D8060.id18241.diff
Mon, Oct 13, 11:21 AM
F18606892: D8060.id18239.diff
Sep 13 2025, 10:41 PM
F18598816: D8060.id.diff
Sep 13 2025, 5:56 AM
F18595901: D8060.diff
Sep 12 2025, 11:06 PM
F18595894: D8060.diff
Sep 12 2025, 11:06 PM
F18592006: D8060.diff
Sep 12 2025, 11:01 AM
F18091788: D8060.id18241.diff
Aug 7 2025, 12:51 AM

Details

Reviewers
epriestley
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPf6e9d36c329c: Legalpad - style NOTE IMPORTANT WARNING remarkup slightly differently
Summary

round them there corners, to create more of a "bubble" effect in legalpad. Ref T3116.

Test Plan

see screenshot, which demonstrates new style works

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

src/applications/legalpad/controller/LegalpadDocumentSignController.php
207โ€“213

I feel like we changed this (or maybe it was javelin includes?) but I can't remember the details.

These should probably be px -- I think the em stuff is mostly old crazy @epriestley leftovers.

(@chad, feel free to yell at us.)

src/applications/legalpad/controller/LegalpadDocumentSignController.php
207โ€“213

Oh, right -- there should be a $this->requireResource(...) now I think.

btrahan updated this revision to Unknown Object (????).Jan 24 2014, 8:45 PM

...i remembered!

require_celerity_resource => $this->requireResource in controllers...

I totally forgot until you made the comment. :P

btrahan updated this revision to Unknown Object (????).Jan 24 2014, 8:47 PM

...and switched to pixels. also made it a little curvier for maximum "bubble-look" for one-line NOTEs

I like px divisible by 4 and 3px on corners. doctordance

webroot/rsrc/css/application/legalpad/legalpad-document.css
17

btw, don't need to do -moz-border-radius, it's been unused just border-radius 22 versions.

http://caniuse.com/border-radius