← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~suutari-olli/openlp/force-split into lp:openlp

 

> My previous review comments on usability have been ignored.
> 
> Please action, as this makes the UI more complicated whilst trying to make
> things better.
> 
> The functionality needs to be merged under a single button.  Change the number
> drop down to add "Auto" to activate you new code using the split button might
> be a better implementation.


I still do not understand what you are trying to suggest.

Currently there are 4 actions available in the song editor:

1. Optional split – Many people are trying to do force splits with this, they do not understand how this works.

2. Verse type: – Combo for selecting the verse type.
3. Verse Number selector: – Select the number for your verse.
4. Insert: – Insert Verse type + Verse Number.

The new “Split” button would be 5th available action.

5. Split – Force split the currently selected verse based on cursor location.
If I understood correctly, you are suggesting I should combine 3 & 5.

I can’t see how this would work. This would just result in
a severe loss of functionality/weird behavior.

I do not think the “Split” button would make the the UI more complicated,
quite the opposite. People keep misunderstanding how “Optional” split 
works and are creating bug reports because they do not understand it. 
By adding the “Split” button, these people would actually have the 
functionality they are looking for, this would also make the 
“Optional Split” more understandable since it would be obviously
a more advanced/different action.

-- 
https://code.launchpad.net/~suutari-olli/openlp/force-split/+merge/296002
Your team OpenLP Core is subscribed to branch lp:openlp.


References