<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />broulik set the repository for this revision to R242 Plasma Frameworks.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D3838" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Use a single global MouseArea and turn delegate into an Item</li>
<li class="remarkup-list-item">Use a single Rectangle for both "today" and "highlighted" state</li>
<li class="remarkup-list-item">Add font.pointSize: -1 workaround in the other labels to avoid runtime warnings</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Hovering and selecting items still works, updates agenda as needed, also "has appointment" overlay still works</li>
<li class="remarkup-list-item">Switching between months updates the highlight - you can see a brief transition from gray to blue and then invisible when you move between the current month and others as the single Rectangle will transition from gray to blue in response to becoming "not today" and opacity 1 to 0 in response to being no longer selected, not too terrible, though.</li>
<li class="remarkup-list-item">Pinch to zoom still works</li>
<li class="remarkup-list-item">didn't notice stray highlight when closing popup with mouse still hovering</li>
</ul>

<p>Unfortunately did not result in a noticeable speedup of first invocation</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Frameworks</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3838" rel="noreferrer">https://phabricator.kde.org/D3838</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/declarativeimports/calendar/qml/DayDelegate.qml<br />
src/declarativeimports/calendar/qml/DaysCalendar.qml</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>