Enhanced weekly widget [WIP]

[JohnnyC1951]

Adds Show Logo, Description and Show URL link and adds abbreviated timezone to show times (avoids confusion across time-zones).

[JohnnyC1951]

I think I doubled up by mistake :( sorry.

[korotkov13]

It's cool feature. I waited so long for it.
I tried it, and found one inconvenience. If I upload a picture(300x300 or other size) it shows full sized in widget. It would be cool if all the pics would be resized automatically to page resolution.
I don't know how to make it. It is just users feedback.
Thanks again for your work.
upd:
one more thing: On desktop it shows very thin. on mobile phone it's ok. and if i press on the show it opens a new(same widget) window.

[JohnnyC1951]

@korotkov13
A revised version attached & footnotes.
I think the problem you had with the link was that you had no url defined in the show url.

Changes:

1. It does not show or have any hyperlinks if the show url field is empty.
2. The width of the show logo is now scaled to 80px wide.
3. Other changes: reduced panel padding by 20px on each side
4. There is inline css if you need to tweak stuff.
5. Be aware that huge numbers of large logo images will slow down the page load.

I would have done another pull request, but I am not authorized to do that (it said when I tried).
weekly-program.phtml.zip

[korotkov13]

Thanks!
One more thing that I did not understand. how to change the time from 12 to 24 hours in format?
It's shows only in 12hours format

upd:
sorry my fault.
everything is working

[JohnnyC1951]

To force it (disable auto locale) change:

"show_start_hour": start_date.toLocaleTimeString([], { hour: 'numeric', minute : 'numeric' }),
"show_end_hour": end_date.toLocaleTimeString([], { hour: 'numeric', minute : 'numeric' }),

To: (or other locale that uses the 24hr clock)

"show_start_hour": start_date.toLocaleTimeString(['en-GB'], { hour: 'numeric', minute : 'numeric' }),
"show_end_hour": end_date.toLocaleTimeString(['en-GB'], { hour: 'numeric', minute : 'n

[JohnnyC1951]

All new version coming very soon. Please do not merge this.

[Robbt]

Ok, the convention is to add [WIP] to the title this will prevent merges. I'll do that for you. Edit the title and remove the [WIP] when it's been updated. Thanks.

[JohnnyC1951]

Thanks Robbt. I had no idea. I had been looking to cancel it somehow too.

[stale]

This issue has been automatically marked as stale because it has not had activity in the last 5 months. It will be closed if no activity occurs in the next month.
Please chat to us on discourse or ask for help on our chat if you have any questions or need further support with getting this issue resolved.
You may also label an issue as pinned if you would like to make sure that it does not get closed by this bot.

[stale]

This issue has been autmatically closed after is was marked as stale and did not receive any further inputs.
Feel free to let us know on discourse or ask for help on our chat if you feel this issue should not have been closed.
Thank you for your contributions.