-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Envelope labels #173
Envelope labels #173
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to work well. I did a bit of manual testing. Here are a few findings:
- One tiny detail: the buttons seem to be too close to each other (this doesn't affect the functionality).
- Then the error message when entering a higher/lower number does not seem to be in Czech.
- With the Skipping first x number of labels maybe preparing the system for the situation when nothing is added (an empty field is sent). At the moment you get an error.
- With the labels sheet itself - is it planned to print out the whole page on the pre-cut labels? And does this layout fit the labels?
if not (0 <= self.skip.data < 16): | ||
flash("Vynechat lze 0 až 15 štítků.", "danger") | ||
return False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this manually and the propper message does not seem to show itself. I see only an info message in English.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works if you have disabled javascript. If you use JS, the browser doesn't allow you to send the request to the server.
Those are messages produced by the browser, I believe. I will test it today on a system with Czech locales.
Good catch, will use 0 in that case.
Yes, exactly, I'll add a comment to the top od the HTML page. |
e13fef2
to
c1036b8
Compare
Nová možnost tisknout štítky na obálky.