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
Unknown Object (File)
Sun, Sep 1, 1:38 AM
Unknown Object (File)
Tue, Aug 27, 11:27 PM
Unknown Object (File)
Mon, Aug 26, 7:56 AM
Unknown Object (File)
Sat, Aug 24, 7:48 AM
Unknown Object (File)
Fri, Aug 23, 1:35 PM
Unknown Object (File)
Thu, Aug 22, 9:48 PM
Unknown Object (File)
Thu, Aug 22, 9:47 PM
Unknown Object (File)
Thu, Aug 22, 9:47 PM

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