Differential D25024 Diff 73 webroot/rsrc/css/layout/phabricator-source-code-view.css

Changeset View

Standalone View

webroot/rsrc/css/layout/phabricator-source-code-view.css

	Show All 19 Lines
	}			}

	.phabricator-source-line {			.phabricator-source-line {
	background-color: {$paste.highlight};			background-color: {$paste.highlight};
	text-align: right;			text-align: right;
	border-right: 1px solid {$paste.border};			border-right: 1px solid {$paste.border};
	color: {$sh-yellowtext};			color: {$sh-yellowtext};
	white-space: nowrap;			white-space: nowrap;
				-webkit-user-select: none;
				user-select: none;
	}			}
				speckUnsubmitted Not Done Inline Actions I think we determined that the only additional line is this WebKit one speck: I think we determined that the only additional line is this WebKit one

	.phabricator-source-line > a::before {			.phabricator-source-line > a::before {
	/* Render the line numbers as a pseudo-element so they don't get copied. */			/* Render the line numbers as a pseudo-element so they don't get copied. */
	content: attr(data-n);			content: attr(data-n);
	}			}

	th.phabricator-source-line a,			th.phabricator-source-line a,
	th.phabricator-source-line span {			th.phabricator-source-line span {
	Show All 32 Lines

	.phabricator-source-code-summary .phabricator-source-code {			.phabricator-source-code-summary .phabricator-source-code {
	white-space: nowrap;			white-space: nowrap;
	}			}


	.phabricator-source-blame-skip,			.phabricator-source-blame-skip,
	.phabricator-source-blame-info {			.phabricator-source-blame-info {
	-moz-user-select: -moz-none;			-moz-user-select: -moz-none;
	-khtml-user-select: none;			-khtml-user-select: none;
	-webkit-user-select: none;			-webkit-user-select: none;
	-ms-user-select: none;			-ms-user-select: none;
	user-select: none;			user-select: none;
				speckUnsubmitted Not Done Inline Actions Should we be using these browser-specific flags or are these ancient? speck: Should we be using these browser-specific flags or are these ancient?
				Leon95AuthorUnsubmitted Done Inline Actions Honestly I have no idea. I am not an css expert. Leon95: Honestly I have no idea. I am not an css expert.
				EkubischtaUnsubmitted Not Done Inline Actions Looks like Safari still requires the vendor prefix Chrome supports without vendor prefix as of about 2016, Edge as of 2018 We should keep the webkit one for sure... Ekubischta: Looks like Safari still requires the vendor prefix {F29362} Chrome supports without vendor…
				speckUnsubmitted Not Done Inline Actions Thanks for checking. My question is aimed at being consistent and not necessarily updating this css here, but making sure the user-select addition above is accounting for the same compatibility. speck: Thanks for checking. My question is aimed at being consistent and not necessarily updating this…
	}			}

	.phabricator-source-blame-skip {			.phabricator-source-blame-skip {
	min-width: 28px;			min-width: 28px;
	border-right: 1px solid {$thinblueborder};			border-right: 1px solid {$thinblueborder};
	}			}

	.phabricator-source-blame-info {			.phabricator-source-blame-info {
	▲ Show 20 Lines • Show All 45 Lines • Show Last 20 Lines