Closed (fixed)
Project:
Office Hours
Version:
5.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
21 Jan 2008 at 23:57 UTC
Updated:
20 Mar 2008 at 20:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Ozeuss commentedDo you mean adding to add several days for the same hours, so that it can be displayed as
sun, mon, thurs 8:00-16:00 , etc....
Right now you can add as many days as you want, because the module will display two additional blank fields (if you need more than 3, just submit the node and re-edit it).
but i think enabling a multiple select is better usability.
I might implement that and other stuff in the next couple of days.
Comment #2
Johannes De Boeck commentedAah, thanks for replying,
It wasn't clear to me that I had to Submit the Node before I could add more days.. wicked..
What I was trying to do is:
Mon 08.00 - 12.00
Tue 08.00 - 17.00
Wen 08.00 - 12.00
Thu 08.00 - 17.00
Fri 08.00 - 17.00
Sat 12.00 - 18.00
Now I can :) Thanks!
(Would be nice though for end-user to be able to do it all in one time..)
Comment #3
scottrigbyit would be great to have 7 fields, and only show however many the user fills out.
Or start with 3, and have an ability to add fields by user as needed.
I've been looking for something to help with this, so looking forward to how it shapes up!
Also happy to test
Comment #4
Anonymous (not verified) commented+1 I would be interested in this as well.
I'm pretty sure that it would be a trivial amount of code to add/edit in order to get some more fields to appear when multiple values is enabled. However, I don't know where to being when doing that as I have remedial programming skills.
txcrew
Comment #5
mnapier commentedHere's what I did to have mine show 7 days on new content and 2 on editing. It's a quick fix at least, you could go into a lot more usability.
File: office_hours.module
Add this:
(change the first $dadd to the number of fields you want for newly created content counting 0 and change the second one for edit.)
Before this:
Change this:
To this:
Comment #6
Johannes De Boeck commentedCool! works like a charm..thank you.
Marking this as patch, unless someone else has a better idea
Comment #7
harry slaughterHere's an actual patch of this fix. I made one change. There will always be a total of 7 entry fields.
Comment #8
mnapier commentedI have cases where I still need the option to have more then 7 days which is why I left it so you'd get two additional days if you edit it. Just FYI.
Comment #9
Ozeuss commentedI'm implementing something similiar to this patch, but instead of changing code manually I've added an option box for the field (for number of widgets showing initially), and a description that explains to the user that if the forms gets submitted, more widgets will show.
Thanks for the input,
I intend to release this fix as part of a stable release of this module either tomorrow or the day after.
Comment #10
Ozeuss commentedMarking as fixed, per new release.
Comment #11
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.