OpeningHoursEditor corrections + enhancements
|Reported by:||Pepou||Owned by:||boman|
|Version:||latest||Keywords:||openinghourseditor plugin enhancement boxes shortcut description dialog list drop-down holiday|
Description (last modified by Don-vip)
Following the discussion in ticket #6584 I've moved a few wishes here:
- While manually editing the text value and noticing a syntax error, the plugin's behavior is highly user-unfriendly - the grid with rectangular boxes disappears and the user may be completely lost.
I'd recommend to keep the graphics part corresponding to the last known correct value (until the text edit field will be corrected) and let the user override the incorrectly edited text part with mouse moves on the graphics part.
- One correction to the description: "You can also create a single time-point (for example: Mo-Fr 09:15) by setting the height of the rectangle to 0, the color of the rectangle will change to green. You can also delete a rectangle by changing its width or height to 0."
The second sentence is misleading. Actually a rectangle can be deleted by a) setting its width to 0, or b) setting its height below 0. Green (single-time point) rectangles can be deleted just with the a) method, because it is not more possible to change their height to anything from 0.
- A couple of times I wanted to add opening hours to some POI. I've opened the plugin GUI ("edit new tag" is selected, but the line next to it is empty), pressed OK, painted some time rectangles (not noticing the Key line keeping empty) and pressed OK, then wondered, where is the new tag gone. Wrong, it was not even created - empty key name.
A much more friendly behavior would be to replace the Key line with the same control, which is used in the simple basic dialog for adding a key + value to object's properties (it is possible to choose from supported values (pull-down with small triangle at the right side) - just as it appears in the plugin's wiki, and/or edit the value manually) and populate the list with common key names (opening_hours, service_times, collection_times, etc. with opening_hours being the default).