mirror of
https://gerrit.wikimedia.org/r/mediawiki/skins/MinervaNeue
synced 2024-11-14 18:05:01 +00:00
2d579183c9
Following up on Jon's POC, this will get rid of the talk board component in favor of linking to the server rendered talk page. Additional Changes: * Cleaned up talk selenium tests. Removed talk_steps.rb which doesn't appear to be used anymore. * Changed talk add button classes to a single class * Moved "Add discussion" button to postheadinghtml per design mock * Added "...talk-explained", "...talk-explained-empty" messages to postheadinghtml per design mock * Due to undesirable jumps in window scroll caused by the section anchor & Toggler.js code when opening the TalkSectionOverlay (read fixme in code), a Promise is always returned from OverlayManager route to reset the scroll position to the top when the section overlay is opened. * Moved "mobile-frontend-talk-fullpage", "mobile-frontend-talk-reply-success", "mobile-frontend-talk-topic-feedback", "mobile-frontend-talk-explained" "mobile-frontend-talk-explained-empty" messages to minerva as minerva is the one who initiates those messages now. * Limited $talk selector to only `.talk` elements since amc talk tab does not need to be targeted * After saving a reply from TalkSectionOverlay, the DOM that is not part of the overlay becomes out of sync since a new reply was created. To get around this, an `onSaveComplete` callback was passed (similar to the TalkSectionAddOverlay) to execute a full page refresh. Although this is clunky, it is the easiest way to resync. Bug: T230695 Depends-On: I80201394fd7015db6700446142b0b4b20829f12e Change-Id: I243f1193bce0da9fa710fc3b5379f90b2d079680
80 lines
1.6 KiB
Plaintext
80 lines
1.6 KiB
Plaintext
@import '../../minerva.less/minerva.variables.less';
|
|
|
|
@divider-line-color: @colorGray14;
|
|
|
|
.minerva-talk-add-button {
|
|
display: block;
|
|
width: 100%;
|
|
max-width: none;
|
|
margin-top: 15px;
|
|
margin-bottom: 20px;
|
|
}
|
|
|
|
.minerva-talk-content-explained {
|
|
// When .skin-minerva--talk-simplified class is not applied to body, we don't
|
|
// want this to show (e.g. on pages without any topics, redundant no content
|
|
// messages could show)
|
|
display: none;
|
|
margin-top: 15px; // setting margin-top in case .minerva-talk-add-button is not present to provide a margin (i.e. when anon user)
|
|
padding-bottom: 15px;
|
|
border-bottom: 1px solid @divider-line-color;
|
|
font-size: @font-size-minerva-small;
|
|
font-weight: bold;
|
|
color: @colorGray5;
|
|
}
|
|
|
|
.minerva-talk-full-page-button {
|
|
display: block;
|
|
position: sticky;
|
|
position: -webkit-sticky;
|
|
bottom: 0;
|
|
left: 0;
|
|
width: 100%;
|
|
padding: 1em;
|
|
color: @colorProgressive;
|
|
background: @colorGray15;
|
|
border-top: 1px solid @divider-line-color;
|
|
text-align: center;
|
|
}
|
|
|
|
// Only run on talk pages
|
|
.client-js .skin-minerva--talk-simplified {
|
|
#toc,
|
|
#mf-section-0,
|
|
.section-heading + div {
|
|
display: none;
|
|
}
|
|
|
|
.minerva-talk-content-explained {
|
|
display: block;
|
|
}
|
|
|
|
.section-heading {
|
|
position: relative;
|
|
border-bottom: 1px solid @divider-line-color;
|
|
font-size: 1em;
|
|
font-family: @fontFamilyBase;
|
|
line-height: 1;
|
|
cursor: pointer;
|
|
display: block;
|
|
margin: 0;
|
|
padding: 0.8em 0;
|
|
|
|
.mw-ui-icon {
|
|
display: none;
|
|
}
|
|
}
|
|
|
|
.return-link {
|
|
margin-bottom: 20px;
|
|
}
|
|
}
|
|
|
|
@media screen and ( min-width: @width-breakpoint-tablet ) {
|
|
.minerva-talk-full-page-button {
|
|
margin-left: auto;
|
|
margin-right: auto;
|
|
max-width: @contentMaxWidthTablet;
|
|
}
|
|
}
|