openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #18081
Re: [Merge] lp:~patrick-zakweb/openlp/bug-936281 into lp:openlp
Does this affect all lists? If so I'm not sure that splitting this merge is
a good idea.
I'll be honest, I am unable to check this branch out to have a look on a
windows machine, but I imagine that this would make distinguishing between
different verses, in the song editor and preview/live controllers, alot
more dificult. (Unless padding was added to the list items)
As we havn't had too many complaints, I don't think we should implement
this without the option of turning it off.
Any chance of getting some "before and after" screen grabs?
Phill
On Nov 20, 2012 6:52 AM, "Raoul Snyman" <
raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx> wrote:
> Sorry Patrik, I have one last request. Can you split this into two merges?
> One without the configuration part (just disable alternate rows on Windows)
> so that we can merge it now, and one with the configuration so that we can
> merge it into post-2.0?
>
> Thanks!
> --
> https://code.launchpad.net/~patrick-zakweb/openlp/bug-936281/+merge/135010
> You are subscribed to branch lp:openlp.
>
--
https://code.launchpad.net/~patrick-zakweb/openlp/bug-936281/+merge/135010
Your team OpenLP Core is subscribed to branch lp:openlp.
Follow ups
References