<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'Bitstream Vera Sans'; font-size:12pt; font-weight:400; font-style:normal;">On Thursday 04 December 2008, asraniel wrote:<br>
> Am Donnerstag 04 Dezember 2008 18.25:57 schrieb Aaron J. Seigo:<br>
> > On Thursday 04 December 2008, Beat Wolf wrote:<br>
> > > This patch brings back the possibility to copy paste the current<br>
> > > selected date out of the calendar. I also had to adapt the scaling,<br>
> > > because the textbox uses vertical space, so the calendar had to use<br>
> > > more horizontal space. If you resize the calendar really badly, you<br>
> > > will see something like in the second screenshot. But this can't be<br>
> > > fixed before 4.3.<br>
> ><br>
> > why can't this be fixed before 4.3?<br>
><br>
> We had a discussion in irc, and it seemed like this won't really be<br>
> possible, because it introduces too many changes to how the calendar is<br>
> rendered. Currently the numbers are pictures, worse, the background boxes<br>
> are the same image as the numbers. So this is really bad.<br>
> What would be needed:<br>
> changing the calendar svg theme<br>
> rendering the numbers directly as text.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>so let's make this clear: it will be harder to change this *after* 4.2 once we ship a theme with the calendar.svg as is.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>anyways ... fixing the number painting can't be difficult: just position them at (box.width() - number.width()) /2. it means more calculations and painting ops, but i doubt it will make a meaningful performance issue.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>can an artist get me an svg with the numbers separate from the background?<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>> > unfortunately this looks fairly ugly, even if it is useful. one solution<br>
> > would be to bring back some other missing bits such as the "go to today"<br>
> > button and week selector (the layout at the bottom could also likely be<br>
> > tweaked with some spacing to make it look good on REALLY wide calendars)<br>
><br>
> pinhero said he wanted to redo the whole layout, but again, for 4.3.<br>
> So in the end i decided that the patch is probably a good start, since in<br>
> it's current state, the calendar is really bad...<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>"really bad" as in "lacks a feature previously there". while in the mythical land of kde those phrases are often used interchangeably, they aren't synonymous.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>* how many people use that feature?<br>
* is putting in something half-assed worse than doing it right later?<br>
* can we put something proper in before then?<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>those are the key questions to determine if "lacks a feature" means "really bad".<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>> But if wanted i can add the "go to today" button and the week selector,<br>
> should be quite easy (well, the week selector needs a new string probably,<br>
> but can be done without a new one, just the numbers)<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>hm. we should hold off on the week selector in that case. but certainly add the go to today button and certainly figure out how to do it so that the text line doesn't stretch out to be a 500px wide ocean of whiteness when the calendar expands.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>perhaps not painting the text edit's background and setting the text colour to white would be sensible, as would centering it beneath the calendar.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>-- <br>
Aaron J. Seigo<br>
humru othro a kohnu se<br>
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>KDE core developer sponsored by Qt Software<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p></body></html>