This module is just a must so thank you very much!
But, why don't put the three field just inline instead of on three distinct rows?
IMHO this layout:

Date From time To time

is more efficient than this:

Date
From time
To time

Best regards,
Saxx

CommentFileSizeAuthor
#4 single-line-display.patch1.43 KBstackpr
#2 date_single_day-901284-2.patch1.63 KBfengtan
#2 date_single_day-901284-2.jpg16.79 KBfengtan

Comments

joachim’s picture

joachim commented

IIRC there's a reason this isn't simple.

If you want to look into it more yourself, I'll review a patch :)

fengtan’s picture

fengtan
Location Montreal, Canada
commented
Status: Active » Needs review
StatusFileSize
new date_single_day-901284-2.jpg16.79 KB
new date_single_day-901284-2.patch1.63 KB

I wrote a patch to put fields inline. Attached is also a screenshot of how it looks. Hope it helps !

joachim’s picture

joachim commented
Status: Needs review » Needs work

Thanks for taking the time to work on a patch.

However, we really shouldn't be using an HTML table for layout... it's not the 1990s any more ;)

stackpr’s picture

stackpr commented
Status: Needs work » Needs review
StatusFileSize
new single-line-display.patch1.43 KB

The attach patch fixes the markup to be able to use the date field CSS. I have the single-line markup end before the "extra fields" are rendered. Thus, timezone and any other fields would go on their own lines below the date+time+time line.

joachim’s picture

joachim commented
Status: Needs review » Fixed

Oh wow.

That's brilliant -- looks much better and no extra CSS needed!

Committed with some whitespace tweaks. Thanks!

Issue #901284 by witti: Changed output of form elements to appear side by side.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.