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
F15281059: D8060.diff
Mon, Mar 3, 3:23 PM
Unknown Object (File)
Fri, Feb 21, 3:38 AM
Unknown Object (File)
Sun, Feb 16, 5:42 AM
Unknown Object (File)
Jan 26 2025, 11:50 AM
Unknown Object (File)
Jan 23 2025, 9:18 PM
Unknown Object (File)
Jan 23 2025, 9:18 PM
Unknown Object (File)
Jan 23 2025, 9:18 PM
Unknown Object (File)
Jan 23 2025, 9:14 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