search

attah / harbour-seaprint

🐟🖨 Network printing for Sailfish OS
GNU General Public License v3.0
16 stars 20 forks source link

SeaPrint 1.2.4: Manuelly setting up the page range is confusing #78

Closed wheelseal closed 1 year ago

wheelseal commented 1 year ago

On the "Printer settings" Page: Point "Page range" (line 508 in translation page): Setting the page range via the slider works fine: Page range 1

But after I do it and tap the stylus, my selected page is not displayed: Page range 2

The "Page Range" hint is not displayed until I enter a number: Page range 3

The hint "page range" should be replaced with "First page" or "Last page" and under the line it should be displayed permanently. Your example numbers 1-12 should also be replace with "First Page" or "Last Page". Alternatively, it could be "Page Area - First Page" or "Page Area - Last Page", which would be even clearer.

attah commented 1 year ago

The position on translation page is irrelevant for reporting UI bugs. It seems i meant to keep input as prefilled value on the keyboard input page; but actually the current behaviour is better. The 1-12 is the accepted range for your specific document, i.e. not an example, and that is really good information. For non-pdfs i'll make sure it says "..." instead of 65535.

Why the bottom label is not displayed until you interact with it, i don't know. That looks like a bug in Jolla's Silica UI stuff and is beyond my control. I will be adding High and Low to the label to make it more clear. But i think it is good that it still says what you are setting too.

The term "Page area" is not understandable.

wheelseal commented 1 year ago

The position on translation page is irrelevant for reporting UI bugs.

I Know but i thought it is easier to find for you ;)

The 1-12 is the accepted range for your specific document, i.e. not an example, and that is really good information.

Yes, that is good information. I misunderstood because I tried it with only one document with 12 pages and mistakenly thought it was meant as an example for all documents.

Why the bottom label is not displayed until you interact with it, i don't know. That looks like a bug in Jolla's Silica UI stuff and is beyond my control.

i Understand.

I will be adding High and Low to the label to make it more clear. But i think it is good that it still says what you are setting too.

Why High and Low? First and Last not better?

The term "Page area" is not understandable.

That´s right, it was a mistake of me. Of course I mean "Page Range".

attah commented 1 year ago

Why High and Low? First and Last not better?

The component is generic to all ranges... so something generic just made more sense to me. (The name of the setting itself is parameterized) Realistically there will be no other ranges most likely, but none the less, that was my reasoning. I think it is good enough... but will think on it some more.