meetinghouse / cms

Laravel CMS
0 stars 3 forks source link

LIGHT THEME: Make banners linkable #237

Open KathJohnson opened 8 years ago

KathJohnson commented 8 years ago

Please make it possible in the interface to attach a link to banners. That is, if a visitor clicks a banner, they will got to some other page or site, wherever the banner leads.

The default should remain that banners are not linked to anything.

This is in response to a request from client New England Soundproofing, received March 8, 2016.

jboz62 commented 8 years ago

@KathJohnson Did we ever find out what the links were on this request? If we get those we can add them via a script, or we can make this a feature in the admin banner section.

meetinghouse commented 8 years ago

I don't have the links for this. I'm not sure they actually exist yet. I suspect it would be cheaper to do this with a script rather than adding it to the interface. This issue should be on hold for now. When I have a list of links ready I'll revive it. Thx.

Sincerely, Katherine Johnson, Research Director Corbett Research Group http://www.restorationtrades.com "Web Marketing that Works"

Publishers of:

The Custom Building & Restoration Trades Directory http://www.restorationtradesdirectory.com

The Building Arts Notebook, "Journal of the Guild of Building Artisans" http://buildingartisansguild.com/

Email: katherine@restorationtrades.com Phone: (413) 475-3154

On Sun, May 1, 2016 at 9:38 PM, jboz62 notifications@github.com wrote:

@KathJohnson https://github.com/KathJohnson Did we ever find out what the links were on this request? If we get those we can add them via a script, or we can make this a feature in the admin banner section.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/meetinghouse/cms/issues/237#issuecomment-216090693

jboz62 commented 8 years ago

@vivek-chaudhari This is a low priority, but a client has asked about it. Would this be difficult to do? Basically, we'd need the ability to add a link to banners, probably via a text box on the admin banner creation page. It would say something like "add URL/Link for your banner:" and a text box to add it. Thoughts?

vivek-chaudhari commented 8 years ago

@jboz62 This is not so difficult, we can achieve it easily. We can provide it as optional. If textbox have value then we will load banner as url, otherwise not.

jboz62 commented 8 years ago

Ok, thanks, Vivek. Let me see if John C. wants to do that.

On Sep 29, 2016, at 12:43 PM, Vivek Chaudhari notifications@github.com wrote:

@jboz62 https://github.com/jboz62 This is not so difficult, we can achieve it easily.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/meetinghouse/cms/issues/237#issuecomment-250523193, or mute the thread https://github.com/notifications/unsubscribe-auth/ALMf8Ak0AbIoWMxzUPn-s8iamFlIU8Ryks5qu-qmgaJpZM4IBUZc.