[DEV] responsive banners

Zelf bezig aan een extensie? Wij kijken graag mee..
Plaats reactie
Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

[DEV] responsive banners

Bericht door Theriddler » 24 feb 2016, 23:01

Extension Name: Responsive Banners
Author: theriddler en John P
Former Author: Cece74


Extensie beschrijving:
phpBB 3.1 extensie module om (reclame) banners te tonen onder het forum in de footer.
Gebasseerd op de ABD extensie code van Nivoslider by cece74 maar met andere events en js code.
Groep Permissies zorgen ervoor dat je de banners alleen aan bepaalde groepen kunt laten zien.

Demo: http://www.peppersrus-forum.nl

Extensie Versie: 1.0.5


Version 1.0.3
***30/02/2016***
+ [Added] Group perrmissions (which group can see banners)

Version 1.0.4
***01/03/2016***
+ [Fix] Group permissions (John P)
+ Large code changes (John P)
+ Code cleanup (John P)

Version 1.0.5
***02/03/2016***
+ [Added] missing {L_COLON} in acp template (dmzx)
- [Removed] Ext.php as it is not neccesary (dmzx)
+ Cleanup adding TABS. (dmzx)

Version 1.0.6
***08/03/2016***
+ [Fix] banners not showing when Enable URL Rewriting was set to off (dmzx)


Extension is still in [DEV] it is not recommended for live forums as it's still in development.
Use at your own risk.


Big credits goed to John P (forumhulp.com) for helping out! and making this a better candidate to be released to the phpbb.com extension database.

Extensie Download:
https://www.phpbbservice.nl/forum/viewt ... 188#p76188


Installatie:
  1. 1. Download het pakket
    2. Pak deze uit.
    4. Upload de map `theriddler` naar `/ext/`
    5. Navigeer in het beheerderspaneel naar `aanpassingen -> beheer extensies`
    6. Zoek in de lijst naar `responsive banners` onder de uitgeschakelde Extensies , en klik op de`inschakelen` link.
    7. Upload banner afbeeldingen via tabblad extensies -> responsivebanners -> beheer
Updaten:
  1. 1. Navigeer in het beheerderspaneel naar `aanpassingen -> beheer extensies`
    2. Zoek in de lijst naar `responsive banners` onder de uitgeschakelde Extensies , en klik op de`uitschakelen` link.
    3. Verwijder de map responsivebanners uit /ext/theriddler
    4. Download het nieuwe pakket
    5. Pak deze uit.
    6. Upload de map `theriddler` naar `/ext/`
    7. Navigeer in het beheerderspaneel naar `aanpassingen -> beheer extensies`
    8. Zoek in de lijst naar `responsive banners` onder de uitgeschakelde Extensies , en klik op de`inschakelen` link.
note:
De maximale breedte van de banner kan geregeld worden in \responsivebanners\styles\all\template\event\overall_footer_after.html

Verander:

Code: Selecteer alles

maxwidth: 468,
wellicht maak ik daar nog een configuratie voor in het acp.
zodat je gewoon een breedte kunt invullen, zonder aanpassingen aan dat bestand.
Laatst gewijzigd door Theriddler op 08 mar 2016, 19:56, 6 keer totaal gewijzigd.
Reden: Extensie Geupdate

Gebruikersavatar
El torro
Berichten: 4165
Lid geworden op: 02 sep 2014, 21:28
Locatie: In het zonnige zuiden
Contacteer:

Re: [DEV] responsive banners

Bericht door El torro » 24 feb 2016, 23:24

Ik neem aan dat het ook zeer wordt afgeraden op deze dev versie op een live forum te gebruiken?

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 25 feb 2016, 09:26

El torro schreef:Ik neem aan dat het ook zeer wordt afgeraden op deze dev versie op een live forum te gebruiken?
In de regel wel inderdaad maar ik kan je verzekeren dat het prima werkt.
De acp code van nivoslider om te uploaden van cece47 heb ik gebruikt als basis voor deze extensie.
Het enige wat veranderd is ,is wat andere css codes, een andere js lib en wat andere events/templates.
zodat de banners onderin responsive zijn en dus verkleint als men op een mobiel of klein scherm kijkt.
Ik kan wel zeggen dat je deze gerust kan installeren.

Feedback hoor ik graag.

een ander idee wat ik nog had is permissie toevoegen zodat je per gebruiker groep kan instellen of men de banner kan zien, denk aan bijv vips die dan geen reclame gaan zien.

John Leeuwis
Berichten: 174
Lid geworden op: 21 mei 2009, 19:24
Locatie: Nijeveen
Contacteer:

Re: [DEV] responsive banners

Bericht door John Leeuwis » 25 feb 2016, 10:18

Theriddler schreef:........... een ander idee wat ik nog had is permissie toevoegen zodat je per gebruiker groep kan instellen of men de banner kan zien, denk aan bijv vips die dan geen reclame gaan zien.
Dat zou de extensie een stúk interessanter maken.
Gewone leden etc. zien de reclame wel en de donateurs zijn daar vrij van.
Gaat zeker werken (voor de kas) :D

Gebruikersavatar
Paul
Beheerder
Beheerder
Berichten: 20297
Lid geworden op: 23 okt 2003, 11:38
Locatie: Utrecht
Contacteer:

Re: [DEV] responsive banners

Bericht door Paul » 25 feb 2016, 12:46

Theriddler schreef:
El torro schreef:Ik neem aan dat het ook zeer wordt afgeraden op deze dev versie op een live forum te gebruiken?
In de regel wel inderdaad maar ik kan je verzekeren dat het prima werkt.
Tja, dat zegt iedere extensie author, ook degene met security issues erin :). Prima werkt betekend niet dat hij veilig is (Ik zeg ook niet dat hij niet veilig is ;)), zolang het extensie team op phpBB.com er niet naar gekeken heeft zou je hem gewoon niet live moeten gebruiken.
Ook zou ik niet zomaar de extensies van cece pakken. Er zijn een groot aantal van zijn extensies afgekeurd door phpBB.com vanwege allerlei issues. Als ik even snel kijk naar jouw ACP module zie ik al dat die van jouw ook niet door de validatie komt :),

Code: Selecteer alles

$target_dir = '' . $phpbb_root_path . 'ext/theriddler/responsivebanners/styles/all/theme/images/';
Extensies mogen geen files uploaden namelijk in hun eigen extensie folder, maar moeten dit doen in een folder die blijft bestaan als extensie is verwijderd, bijvoorbeeld store/.

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 25 feb 2016, 18:24

thnx paul voor de feedback en duidelijk, ik ga er naar kijken.

@John ook daar ga ik binnenkort mee aan de slag. staat op mijn todo lijstje :)

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 01 mar 2016, 13:52

*** Extensie Geupdate

Gebruikersavatar
dmzx
Berichten: 46
Lid geworden op: 25 jan 2015, 07:58
Locatie: Zuid-Holland
Contacteer:

Re: [DEV] responsive banners

Bericht door dmzx » 01 mar 2016, 23:07

In "version": "1.0.4"

ext.php? + * Extension Quickstart Package.

In acp_responsivebanners_body.html gebruikt {L_COLON} voor :

In listener.php gebruik TAB key ;)

In permissions_responsivebanners.php EN en NL staat:

Code: Selecteer alles

/**
*
* BB Points extension for the phpBB Forum Software package.
*
* @copyright (c) 2015 ForumHulp.com <http://forumhulp.com>
* @license Proprietary
*
*/
grtz dmzx
Afbeelding

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 02 mar 2016, 16:46

dmzx schreef:In "version": "1.0.4"

ext.php? + * Extension Quickstart Package.

In acp_responsivebanners_body.html gebruikt {L_COLON} voor :

In listener.php gebruik TAB key ;)

In permissions_responsivebanners.php EN en NL staat:

Code: Selecteer alles

/**
*
* BB Points extension for the phpBB Forum Software package.
*
* @copyright (c) 2015 ForumHulp.com <http://forumhulp.com>
* @license Proprietary
*
*/
grtz dmzx
Thnx dmzx voor het reviewen, de problemen/punten die je aangaf zijn inmiddels verholpen. een update komt er zo spoedig aan.
V 1.0.5 ;)
Extensie Geupdate naar 1.0.5
- Theriddler

Gebruikersavatar
Solidjeuh
Berichten: 681
Lid geworden op: 05 okt 2010, 06:28
Locatie: Aalst / Belgie
Contacteer:

Re: [DEV] responsive banners

Bericht door Solidjeuh » 05 mar 2016, 22:31

Bedankt! Ik had dit al op mijn forum, waar was geen extensie..
Is veel makkelijker nu!

Juist 1 klein vraagje, is het ook mogelijk om de banners random te tonen?
Ik had dit in het script toegevoegd ( startSlide: "shuffle" )maar hoe kan ik dit in jouw extensie toevoegen of wijzigen?

En volgens mij mist deze vertaling nog in het ACP bestandje:

Code: Selecteer alles

	'TITLE'							        => 'Titel',
Nogmaals bedankt voor de extensie! :D
Forum voor NL Extensie vertalingen ---> https://www.solidjeuh.fun
----------------------------------------------------------------------------

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 07 mar 2016, 14:25

Solidjeuh schreef:Bedankt! Ik had dit al op mijn forum, waar was geen extensie..
Is veel makkelijker nu!

Juist 1 klein vraagje, is het ook mogelijk om de banners random te tonen?
Ik had dit in het script toegevoegd ( startSlide: "shuffle" )maar hoe kan ik dit in jouw extensie toevoegen of wijzigen?
Ja dat is mogelijk.

open: \responsivebanners\styles\all\template\event\overall_footer_after.html

Zoek:

Code: Selecteer alles

auto: true,
Plaats daaronder op een nieuwe regel:

Code: Selecteer alles

random: true,
Vergeet niet de forum cache te legen wanneer je het bestand terug upload.

Meer opties vind je hier:
http://responsiveslides.com/

kijk bij: 5. Options you can customize

Wellicht in een latere versie maak ik daar ook nog aan en uit schakelaars voor zodat je helemaal geen code hoeft aan te passen. ;)

Solidjeuh schreef:En volgens mij mist deze vertaling nog in het ACP bestandje:

Code: Selecteer alles

	'TITLE'							        => 'Titel',
Nogmaals bedankt voor de extensie! :D
Graag gedaan en Thnx ga ik onderzoeken.

Gebruikersavatar
Solidjeuh
Berichten: 681
Lid geworden op: 05 okt 2010, 06:28
Locatie: Aalst / Belgie
Contacteer:

Re: [DEV] responsive banners

Bericht door Solidjeuh » 07 mar 2016, 20:37

Super! daar had ik dus niet gekeken :oops: :lol:

== EDIT ==

Had ik dus wel gekeken, maar ik had 'shuffle' gedaan ipv true
Daarom dat het dus niet werkte :)
Forum voor NL Extensie vertalingen ---> https://www.solidjeuh.fun
----------------------------------------------------------------------------

Gebruikersavatar
dmzx
Berichten: 46
Lid geworden op: 25 jan 2015, 07:58
Locatie: Zuid-Holland
Contacteer:

Re: [DEV] responsive banners

Bericht door dmzx » 08 mar 2016, 18:53

Linken op het forum met app.php geven de banner niet weer. (Enable URL Rewriting Uit)

Zie: Image

Verander in listener.php onderstaande:

Code: Selecteer alles

public function add_page_footer_slider($event)
	{
		if ($this->auth->acl_get('u_viewbanners'))
		{
			$board_url = generate_board_url();
			
			$sql = 'SELECT file_name, img_alt, img_title, img_url FROM ' . $this->tr_banners_table . ' ORDER BY order_img';
			$result = $this->db->sql_query($sql);
			while ($row = $this->db->sql_fetchrow($result))
			{
				$this->template->assign_block_vars('banners', array(
					'SLIDE_IMG'      => $board_url .'/images/responsivebanners/'. $row['file_name'],
					'IMG_ALT'		=> $row['img_alt'],
					'IMG_TITLE'		=> $row['img_title'],
					'IMG_URL'		=> $row['img_url']
				));
			}
			$this->db->sql_freeresult($result);
		}
	}
De $board_url = generate_board_url(); is toegevoegd

grtz
Afbeelding

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 08 mar 2016, 19:30

dmzx schreef:Linken op het forum met app.php geven de banner niet weer. (Enable URL Rewriting Uit)

Zie: Image

Verander in listener.php onderstaande:

Code: Selecteer alles

public function add_page_footer_slider($event)
	{
		if ($this->auth->acl_get('u_viewbanners'))
		{
			$board_url = generate_board_url();
			
			$sql = 'SELECT file_name, img_alt, img_title, img_url FROM ' . $this->tr_banners_table . ' ORDER BY order_img';
			$result = $this->db->sql_query($sql);
			while ($row = $this->db->sql_fetchrow($result))
			{
				$this->template->assign_block_vars('banners', array(
					'SLIDE_IMG'      => $board_url .'/images/responsivebanners/'. $row['file_name'],
					'IMG_ALT'		=> $row['img_alt'],
					'IMG_TITLE'		=> $row['img_title'],
					'IMG_URL'		=> $row['img_url']
				));
			}
			$this->db->sql_freeresult($result);
		}
	}
De $board_url = generate_board_url(); is toegevoegd

grtz
Dat heb ik niet meegenomen in mijn test, ik had app.php uitstaan. :roll:
Bedankt voor de toevoeging/verbetering dmzx. Zal deze zo snel mogelijk toevoegen.

Edit:
Extensie geupdate V1.0.6
- Theriddler

Gebruikersavatar
Froddelaar
Berichten: 272
Lid geworden op: 29 mar 2016, 15:09
Locatie: Aalst, Belgie
Contacteer:

Re: [DEV] responsive banners

Bericht door Froddelaar » 12 aug 2017, 20:41

Enkele foutjes in de vertaling

Code: Selecteer alles

'De %s is met success geladen.',
veranderen naar:
'De %s is met succes geladen.',

Code: Selecteer alles

'Sorry, het bestand bestaat rreds.',
veranderen naar:
'Sorry, het bestand bestaat reeds.',

Code: Selecteer alles

'Illegal type: Bestand is geen afbeelding.',
verandern naar:
'Illegaal type: Bestand is geen afbeelding.',

Code: Selecteer alles

'bestand %s os met success verwijderd.',
veranderen naar:
'bestand %s is met succes verwijderd.',
Ik probeer ook mijn website html5 te maken.
Maar als ik bij deze extensie een css regel aanmaak om het te centreren werkt het niet.
De banner komt niet perfect in het midden te staan.. weet jij een oplossing?

Code: Selecteer alles

.customcenter {
    text-align: center;
}
Wij promoten UW muziek in ons forum & delen alles via Sociale media!
Muziek wordt ook toegevoegd in de playlist van
textradio.be!
Mail uw single + hoesje + info naar: info@muziekpromo.net of Solidjeuh@textradio.be
Onze Website: https://www.muziekpromo.net

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 21 aug 2017, 07:23

zo te zien is het je al gelukt ? zoniet laat het dan nog even weten.

Gebruikersavatar
Froddelaar
Berichten: 272
Lid geworden op: 29 mar 2016, 15:09
Locatie: Aalst, Belgie
Contacteer:

Re: [DEV] responsive banners

Bericht door Froddelaar » 21 aug 2017, 13:28

Theriddler schreef:
21 aug 2017, 07:23
zo te zien is het je al gelukt ? zoniet laat het dan nog even weten.
Neen, ik heb de standaard <center> terug geplaatst, want lukte me niet..
Wij promoten UW muziek in ons forum & delen alles via Sociale media!
Muziek wordt ook toegevoegd in de playlist van
textradio.be!
Mail uw single + hoesje + info naar: info@muziekpromo.net of Solidjeuh@textradio.be
Onze Website: https://www.muziekpromo.net

Gebruikersavatar
Theriddler
Berichten: 1103
Lid geworden op: 20 apr 2012, 07:47
Contacteer:

Re: [DEV] responsive banners

Bericht door Theriddler » 22 aug 2017, 08:08

Froddelaar schreef:
21 aug 2017, 13:28
Theriddler schreef:
21 aug 2017, 07:23
zo te zien is het je al gelukt ? zoniet laat het dan nog even weten.
Neen, ik heb de standaard <center> terug geplaatst, want lukte me niet..
Als je de <center> weghaald
en in het bestand: /ext/theriddler/responsivebanners/styles/all/theme/responsiveslides.css

het volgende veranderd:

Code: Selecteer alles

.rslides_container {
  position: relative;
  /*float: left;*/
  width: 100%;
  /*! max-width: 900px; */
  }
Naar:

Code: Selecteer alles

.rslides_container {
  position: relative;
  width: 100%;
  }
Zou het gecentreerd moeten zijn.

Gebruikersavatar
Froddelaar
Berichten: 272
Lid geworden op: 29 mar 2016, 15:09
Locatie: Aalst, Belgie
Contacteer:

Re: [DEV] responsive banners

Bericht door Froddelaar » 22 aug 2017, 11:52

Yeps dat werkt! Bedankt!
Het is nu ook html5 valid :)
Wij promoten UW muziek in ons forum & delen alles via Sociale media!
Muziek wordt ook toegevoegd in de playlist van
textradio.be!
Mail uw single + hoesje + info naar: info@muziekpromo.net of Solidjeuh@textradio.be
Onze Website: https://www.muziekpromo.net

Gebruikersavatar
nl1sms
Berichten: 818
Lid geworden op: 12 mei 2005, 16:11
Locatie: Zaanstad

Re: [DEV] responsive banners

Bericht door nl1sms » 30 sep 2017, 10:20

Leuke extensie. Ik kan alleen nergens de hoogte van een banner instellen en ik kan geen banner in de overall_header krijgen ?!?! Hij zet ze alleen maar in de footer ??? kan ik dit op de een of andere manier fixxen? thx

edit: ik ben bang dat ik niets aan deze extensie heb :( meerdere banners slide hij en ik wil er gewoon 4 onder elkaar hebben zonder slide.

edit2: als je de extensie uitschakelt blijft hij zichtbaar in de ACP bij extensies:
ACP_RESPONSIVE_BANNERS

ACP_RESPONSIVE_BANNERS_MANAGE

Plaats reactie