Hi Adam, k - fonts-cantarell was maybe not a good idea.
Have changed it in the recent upload to a standard Sans font which looks ok on stretch xfce (well at least to my eyes) - but as you mentioned its configurable. Have raised the issue upstream about slick-greeter.conf - I'll do an upstream PR anyway very soon to cover this and therefore will be made available in a subsequent update. I had a look at the configurator - it currently has a dependency according to the upstream debian/control file on a package called python3-xapp that doesn't exist in Debian. I need to have a closer look at the source to see it really does need this dependency. Assuming I can iron this bit out, yes I intend to package it as well. Yes please to New if you are happy with the minor tweaks I've done above. David On 24 May 2017 at 21:34, Adam Borowski <kilob...@angband.pl> wrote: > On Wed, May 24, 2017 at 06:57:17PM +0100, foss.freedom wrote: >> I have switched liblightdm-gobject-1-dev and liblightdm-gobject-dev >> around in the latest upload. Hopefully this will resolve the first >> alternate rule on buildds > > Builds and runs fine, thanks! > > The package is in a shape fit for unstable now; let's continue with review > but you can choose whether you prefer to get it into NEW then do incremental > improvements, or to fix the issues immediately. > > * the fonts looks ugly. I see you want to use fonts-cantarell but don't > actually depend on it; since it's configurable, a Suggests: or Recommends: > stanza would be appropriate. > > * the README talks about /etc/lightdm/slick-greeter.conf -- yet it 1. is not > shipped with the package and 2. there's not a word of documentation about > what can you put in that file and what is its syntax. I'd recommend > installing such a file with commented out possible settings. > > * it talks about a configurator, "lightdm-settings". I assume someone will > package it in nearby future, right? > > > Meow! > -- > Don't be racist. White, amber or black, all beers should be judged based > solely on their merits. Heck, even if occasionally a cider applies for a > beer's job, why not? > On the other hand, corpo lager is not a race.