revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-23 23:32:40 +01:00
Code Issues Releases Wiki Activity
phorge-phorge/webroot/rsrc/css/application/base/standard-page-view.css

102 lines
1.8 KiB
CSS
Raw Normal View History

Distribute Celerity includes.
2011-01-25 20:31:40 +01:00
/**
* @provides phabricator-standard-page-view
*/
.phabricator-standard-page {
background: #ffffff;
}
Add local navigation to Differential Summary: Adds a flexible navigation menu to diffs that shows you your current position in the diff. Anticipating some "this is the best thing ever" and some "this is the wosrt thing ever" on this, but let's see how much pushback we get? It seems pretty good to me. Test Plan: Will attach screenshots. Reviewers: vrana, btrahan Reviewed By: vrana CC: aran Maniphest Tasks: T1633, T1591 Differential Revision: https://secure.phabricator.com/D3355
2012-08-22 00:01:20 +02:00
.phabricator-anchor-view,
.phabricator-anchor-navigation-marker {
position: absolute;
margin-top: -15px;
}
Countdown tweaks Summary: A few tweaks to hsb's Countdown implementation: - Allow the page to be rendered "chromeless", suitable for display on one of the dozens of monitors everyone has laying around. - Show title of countdown in deletion dialog. - When creating a new countdown default to time(), not Dec 31, 1969. - Add extra "/" after editing to avoid needless redirect. - Tweak some page titles. - Show countdown author in list view. - Highlight tab in list view. - Tweak menu copy. - Link countdown title in list view, separate buttons into different columns so they pick up padding. Test Plan: Created, edited and deleted a timer. Viewed a timer and toggled chrome mode. Viewed timer list. Reviewed By: hsb Reviewers: hsb, aran, jungejason, tuomaspelkonen CC: aran, hsb, epriestley Differential Revision: 454
2011-06-14 02:35:13 +02:00
.phabricator-chromeless-page .phabricator-standard-page {
background: transparent;
border-width: 0px;
Allow installs to add a custom corp/org header link Summary: A bunch of installs are doing this to varying degrees of success anyway, make it easier and nudge them toward a more consistent approach. Test Plan: Set a custom logo, viewed normal and admin pages. Reviewers: btrahan Reviewed By: btrahan CC: aran, epriestley Maniphest Tasks: T700 Differential Revision: https://secure.phabricator.com/D2019
2012-03-26 18:29:31 +02:00
}
Provide basic structure for keyboard shortcuts Summary: Implements a simple infrastructure for keyboard shortcuts, see T184, and a "help" shortcut. There's a lot of room for refinement here but I think it basically works. Each shortcut can also provide a "tooltip" handler which allows it to show help when the alt/option key is held down. Test Plan: Pressed "?" and got help. Pressed "?" in various contexts where it should not activate (modifier keys, text input focused) and didn't get help. Reviewers: aran, tuomaspelkonen, jungejason CC: moskov Differential Revision: 362
2011-05-28 20:36:00 +02:00
.keyboard-shortcut-help td,
.keyboard-shortcut-help th {
padding: 8px;
vertical-align: middle;
}
.keyboard-shortcut-help th {
white-space: nowrap;
color: #666666;
}
.keyboard-shortcut-help kbd {
background: #222222;
padding: 6px;
color: #ffffff;
font-weight: bold;
border: 1px solid #555555;
}
Provide basic keyboard navigation support for Differential. Summary: ReviewBoard has a fancier version of this feature that's more granular -- the keyboard can focus on individual changes. I think that's good and intend to implement something similar, but this gets us a step closer and gets rid of some of the bookkeeping stuff like making shortcuts discoverable. (I have another brnach with Maniphest merging which also uses fatcow icons, which is why the README seems a little out of context.) Test Plan: Used "j" and "k" to jump between changesets. Pressed "?" and got a list of available shortcuts. Reviewed By: tuomaspelkonen Reviewers: aran, jungejason, tuomaspelkonen CC: moskov, aran, epriestley, tuomaspelkonen Differential Revision: 412
2011-06-08 20:53:10 +02:00
.keyboard-focus-focus-reticle {
Improve visibility of the keyboard shortcut focus reticle Summary: This ended up being pretty hard to see, make it a bit easier. Test Plan: Focused things using the keyboard reticle. Reviewed By: tomo Reviewers: tomo, moskov, jungejason, tuomaspelkonen, aran CC: aran, epriestley, tomo Differential Revision: 483
2011-06-20 23:21:17 +02:00
background: #ffffd3;
Provide basic keyboard navigation support for Differential. Summary: ReviewBoard has a fancier version of this feature that's more granular -- the keyboard can focus on individual changes. I think that's good and intend to implement something similar, but this gets us a step closer and gets rid of some of the bookkeeping stuff like making shortcuts discoverable. (I have another brnach with Maniphest merging which also uses fatcow icons, which is why the README seems a little out of context.) Test Plan: Used "j" and "k" to jump between changesets. Pressed "?" and got a list of available shortcuts. Reviewed By: tuomaspelkonen Reviewers: aran, jungejason, tuomaspelkonen CC: moskov, aran, epriestley, tuomaspelkonen Differential Revision: 412
2011-06-08 20:53:10 +02:00
position: absolute;
Improve visibility of the keyboard shortcut focus reticle Summary: This ended up being pretty hard to see, make it a bit easier. Test Plan: Focused things using the keyboard reticle. Reviewed By: tomo Reviewers: tomo, moskov, jungejason, tuomaspelkonen, aran CC: aran, epriestley, tomo Differential Revision: 483
2011-06-20 23:21:17 +02:00
border: 1px solid #999900;
Provide basic keyboard navigation support for Differential. Summary: ReviewBoard has a fancier version of this feature that's more granular -- the keyboard can focus on individual changes. I think that's good and intend to implement something similar, but this gets us a step closer and gets rid of some of the bookkeeping stuff like making shortcuts discoverable. (I have another brnach with Maniphest merging which also uses fatcow icons, which is why the README seems a little out of context.) Test Plan: Used "j" and "k" to jump between changesets. Pressed "?" and got a list of available shortcuts. Reviewed By: tuomaspelkonen Reviewers: aran, jungejason, tuomaspelkonen CC: moskov, aran, epriestley, tuomaspelkonen Differential Revision: 412
2011-06-08 20:53:10 +02:00
}
Tweak style on "Create Another Task" button Summary: Not totally sure I'm in love with this but I think it's somewhat non-terrible, despite the lack of lens flare. Also made "Cancel" take you back to the task if you got to "Create" from "Create Another Task". Test Plan: - Style: https://secure.phabricator.com/file/view/PHID-FILE-ad37d3c1f3b2c7a7a7d1/ - Hit "Cancel" from "Create Another", got sent back to task. - Hit "Cancel" from normal create, got sent back to list. - Tried to save an invalid task after making changes to CC/Projects, changes were preserved. Reviewed By: codeblock Reviewers: hunterbridges, jungejason, tuomaspelkonen, aran, codeblock CC: aran, epriestley, codeblock Differential Revision: 736
2011-07-27 19:46:22 +02:00
.workflow-header {
background: #efefef;
padding: 6px 2em;
text-align: right;
margin-bottom: 6px;
border-bottom: 1px solid #dfdfdf;
}
.workflow-header button {
float: right;
}
Add object status to Handles Summary: We use ObjectHandles as proxy objects which can refer to any other object in the system. Add the concept of the underlying object's "status" (e.g., open, closed or busy). This allows us to render completed tasks and revisions with strikethrough. In the future, if we implement OOO or something, we could render users with a "busy" status if they're on vacation, etc. Test Plan: Viewed a task with closed revisions and dependencies: https://secure.phabricator.com/file/view/PHID-FILE-6183e81286fa3288d33d/ Reviewed By: codeblock Reviewers: codeblock, hunterbridges, jungejason, tuomaspelkonen, aran CC: aran, codeblock Differential Revision: 772
2011-08-03 15:37:18 +02:00
Fix link colors. Summary: Fix the visited state on visited closed tasks. Test Plan: Create and close tasks, visit them, note state doesnt change. Check hover states. Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5298
2013-03-08 19:43:20 +01:00
a.handle-status-closed {
Add object status to Handles Summary: We use ObjectHandles as proxy objects which can refer to any other object in the system. Add the concept of the underlying object's "status" (e.g., open, closed or busy). This allows us to render completed tasks and revisions with strikethrough. In the future, if we implement OOO or something, we could render users with a "busy" status if they're on vacation, etc. Test Plan: Viewed a task with closed revisions and dependencies: https://secure.phabricator.com/file/view/PHID-FILE-6183e81286fa3288d33d/ Reviewed By: codeblock Reviewers: codeblock, hunterbridges, jungejason, tuomaspelkonen, aran CC: aran, codeblock Differential Revision: 772
2011-08-03 15:37:18 +02:00
text-decoration: line-through;
Make closed tasks appear grey Summary: In long list of tasks, its hard to differentiate between open and closed tasks. This adds some color in addition to the strikethrough. Hover will still be blue. I'm open to other suggestions as well if you don't think this is needed. Test Plan: Test Unbeta Pholio Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5270
2013-03-07 17:16:13 +01:00
color: #676767;
}
Fix link colors. Summary: Fix the visited state on visited closed tasks. Test Plan: Create and close tasks, visit them, note state doesnt change. Check hover states. Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5298
2013-03-08 19:43:20 +01:00
a.handle-status-closed:hover {
Make closed tasks appear grey Summary: In long list of tasks, its hard to differentiate between open and closed tasks. This adds some color in addition to the strikethrough. Hover will still be blue. I'm open to other suggestions as well if you don't think this is needed. Test Plan: Test Unbeta Pholio Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5270
2013-03-07 17:16:13 +01:00
text-decoration: line-through;
color: #19558D;
Add object status to Handles Summary: We use ObjectHandles as proxy objects which can refer to any other object in the system. Add the concept of the underlying object's "status" (e.g., open, closed or busy). This allows us to render completed tasks and revisions with strikethrough. In the future, if we implement OOO or something, we could render users with a "busy" status if they're on vacation, etc. Test Plan: Viewed a task with closed revisions and dependencies: https://secure.phabricator.com/file/view/PHID-FILE-6183e81286fa3288d33d/ Reviewed By: codeblock Reviewers: codeblock, hunterbridges, jungejason, tuomaspelkonen, aran CC: aran, codeblock Differential Revision: 772
2011-08-03 15:37:18 +02:00
}
Actually apply monospacing to the monospaced font preference example Summary: See T551. We don't apply the default monospacing rules to the example, so if you don't have a custom font selection you don't see the default accurately. Test Plan: Deleted my preference, saw an accurate default. Set my preference to "14px impact", ensured it was respected in applications. Reviewers: jungejason, nh, tuomaspelkonen, aran Reviewed By: nh CC: aran, nh Differential Revision: 1035
2011-10-23 22:36:44 +02:00
Use black bullet instead of gray background for disabled users Summary: Just because I like it more. Test Plan: View diff with comment from disabled user. Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Koolvin Differential Revision: https://secure.phabricator.com/D2525
2012-05-21 23:54:39 +02:00
a.handle-disabled,
Highlight away status by bullet and display it everywhere Summary: D2484#13 Test Plan: Display revision list. Display revision detail. Reviewers: epriestley Reviewed By: epriestley CC: btrahan, aran, Koolvin Differential Revision: https://secure.phabricator.com/D2501
2012-05-19 18:18:34 +02:00
a.handle-status-away,
a.handle-status-sporadic {
padding-left: 12px;
background-repeat: no-repeat;
background-position: -4px center;
Highlight away and sporadic users in revision list Summary: This is not so general as `getRequiredHandlePHIDs()`. It allows bulk loading of user statuses only in revision list. It also loads data in `render()`. I'm not sure if it's OK. Maybe we can use the colorful point here. Or maybe some unicode symbol? Test Plan: {F11451, size=full} Reviewers: btrahan, epriestley Reviewed By: btrahan CC: aran, Koolvin Differential Revision: https://secure.phabricator.com/D2484
2012-05-17 03:42:06 +02:00
}
Highlight away status by bullet and display it everywhere Summary: D2484#13 Test Plan: Display revision list. Display revision detail. Reviewers: epriestley Reviewed By: epriestley CC: btrahan, aran, Koolvin Differential Revision: https://secure.phabricator.com/D2501
2012-05-19 18:18:34 +02:00
a.handle-status-away {
background-image: url(/rsrc/image/icon/fatcow/bullet_red.png);
}
a.handle-status-sporadic {
background-image: url(/rsrc/image/icon/fatcow/bullet_orange.png);
Highlight away and sporadic users in revision list Summary: This is not so general as `getRequiredHandlePHIDs()`. It allows bulk loading of user statuses only in revision list. It also loads data in `render()`. I'm not sure if it's OK. Maybe we can use the colorful point here. Or maybe some unicode symbol? Test Plan: {F11451, size=full} Reviewers: btrahan, epriestley Reviewed By: btrahan CC: aran, Koolvin Differential Revision: https://secure.phabricator.com/D2484
2012-05-17 03:42:06 +02:00
}
Improve Differential handling of disabled users Summary: We currently allow you to assign code review to disabled users, but should not. Test Plan: - Created revisions with no reviewers and only disabled reviewers, was appropriately warned. - Looked at a disabled user handle link, was clearly informed. - Tried to create a new revision with a disabled reviewer, was rebuffed. Reviewers: btrahan, jungejason Reviewed By: btrahan CC: aran Differential Revision: https://secure.phabricator.com/D1429
2012-01-17 01:54:05 +01:00
a.handle-disabled {
Use black bullet instead of gray background for disabled users Summary: Just because I like it more. Test Plan: View diff with comment from disabled user. Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Koolvin Differential Revision: https://secure.phabricator.com/D2525
2012-05-21 23:54:39 +02:00
background-image: url(/rsrc/image/icon/fatcow/bullet_black.png);
Improve Differential handling of disabled users Summary: We currently allow you to assign code review to disabled users, but should not. Test Plan: - Created revisions with no reviewers and only disabled reviewers, was appropriately warned. - Looked at a disabled user handle link, was clearly informed. - Tried to create a new revision with a disabled reviewer, was rebuffed. Reviewers: btrahan, jungejason Reviewed By: btrahan CC: aran Differential Revision: https://secure.phabricator.com/D1429
2012-01-17 01:54:05 +01:00
}
Make it harder to miss errors and warnings while developing Phabricator Summary: If a page generates warnings or errors, you only get a little red dot in DarkConsole which is hard to see. DarkConsole is also fairly big and there are plenty of reasons not to leave it open all the time. Instead, unconditionally show a big message to developers if there are errors or warnings. We could make this more sophisticated eventually, but the value is just that you see it. Test Plan: Browsed pages with and without warnings, got the right banner state. Reviewers: nh, btrahan, jungejason Reviewed By: btrahan CC: aran, btrahan Maniphest Tasks: T734 Differential Revision: https://secure.phabricator.com/D1307
2012-01-04 16:35:52 +01:00
.aphront-developer-error-callout {
Show a warning banner to administrators when there are unresolved setup warnings Summary: We now issue only valid setup warnings, so we can let administrators know when we detect problems. Test Plan: Banner: {F29568} Created a fake issue; saw banner. No banner inside /config/. Resovled the issue, banner went away. Reviewers: btrahan, chad Reviewed By: chad CC: aran Maniphest Tasks: T2221 Differential Revision: https://secure.phabricator.com/D4493
2013-01-18 01:25:47 +01:00
position: relative;
Make it harder to miss errors and warnings while developing Phabricator Summary: If a page generates warnings or errors, you only get a little red dot in DarkConsole which is hard to see. DarkConsole is also fairly big and there are plenty of reasons not to leave it open all the time. Instead, unconditionally show a big message to developers if there are errors or warnings. We could make this more sophisticated eventually, but the value is just that you see it. Test Plan: Browsed pages with and without warnings, got the right banner state. Reviewers: nh, btrahan, jungejason Reviewed By: btrahan CC: aran, btrahan Maniphest Tasks: T734 Differential Revision: https://secure.phabricator.com/D1307
2012-01-04 16:35:52 +01:00
padding: 2em;
background: #aa0000;
color: white;
text-align: center;
font-size: 11px;
font-family: "Verdana";
}
Show a warning banner to administrators when there are unresolved setup warnings Summary: We now issue only valid setup warnings, so we can let administrators know when we detect problems. Test Plan: Banner: {F29568} Created a fake issue; saw banner. No banner inside /config/. Resovled the issue, banner went away. Reviewers: btrahan, chad Reviewed By: chad CC: aran Maniphest Tasks: T2221 Differential Revision: https://secure.phabricator.com/D4493
2013-01-18 01:25:47 +01:00
.setup-warning-callout {
position: relative;
Make ignored setup issues grey Summary: When I ignore setup issues, I want them to look dealt with, and keep yellow for new ones. Also updated callout colors. Test Plan: Ignored a number of issues. Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D6300
2013-06-25 19:17:46 +02:00
padding: 8px 16px;
background: {$lightyellow};
Show a warning banner to administrators when there are unresolved setup warnings Summary: We now issue only valid setup warnings, so we can let administrators know when we detect problems. Test Plan: Banner: {F29568} Created a fake issue; saw banner. No banner inside /config/. Resovled the issue, banner went away. Reviewers: btrahan, chad Reviewed By: chad CC: aran Maniphest Tasks: T2221 Differential Revision: https://secure.phabricator.com/D4493
2013-01-18 01:25:47 +01:00
}
Copy permalink