search

jonge-democraten / website

JD website
https://jongedemocraten.nl
MIT License
6 stars 2 forks source link

Remove twitter code #203

Closed mathemaat closed 6 years ago

mathemaat commented 6 years ago

zie titel

mathemaat commented 6 years ago

Hey @bartromgens

Kun je even meedenken? Hoe precies zouden wij de CSS van de Twitterfeed moeten nabootsen? De designer heeft hier natuurlijk niks voor gemaakt.

Als het één op één moet zijn, dan moet ik nog de volgende dingen doen:

Vooral die bovenste twee punten lijken me veel/ingewikkeld werk voor weinig winst.

Dit is wat ik tot nu toe heb. De bovenste is mijn versie. De onderste is de originele Twitterfeed. Daar weer onder zie je hoe het in de huidige website is opgelost.

afbeelding

afbeelding

Wat zou je minimaal nog toegevoegd willen zien aan de versie die ik gemaakt heb?

mathemaat commented 6 years ago

Deze WIP-versie is eventueel uit te checken via branch 203-custom-twitter-feed.

secretarisjongedemocraten commented 6 years ago

Als er problemen zijn bij de uitvoer van het design kunnen we ook de designer nog om hulp vragen. Er zit wel enige nazorg in het contract namelijk.

Met vriendelijke groet,

Pauline Gerth van Wijk,

Secretaris Jonge Democraten

T: +31 (0)6 16327311 E: secretaris@jongedemocraten.nl W: www.jongedemocraten.nl

2017-10-30 21:58 GMT+01:00 mathemaat notifications@github.com:

Deze WIP-versie is eventueel uit te checken via branch 203-custom-twitter-feed.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jonge-democraten/website/issues/203#issuecomment-340582331, or mute the thread https://github.com/notifications/unsubscribe-auth/AJrDlslMMcIRPvFNHwo08ZbiJ52aHXuxks5sxjh1gaJpZM4QB3uv .

bartromgens commented 6 years ago

@mathemaat Ik vind het zelf zo al prima, maar dit lijkt me een beslissing voor Pauline. Dan kunnen we eventueel vragen de designers het widget te laten maken.

mathemaat commented 6 years ago

Interessant dat Pauline actief is onder @bashazeborg . Is dat een account dat van secretaris op secretaris wordt doorgegeven?

@bashazeborg (Pauline,) welke zaken mis jij nog in mijn versie van de Twitterfeed (zie hierboven)? Denk aan acties als reply, retweet en favorite, zoals ook in de huidige site.

Als je mijn versie wat saai of bleekjes vindt, wat ik me kan voorstellen aangezien ik die kekke icooontjes van Twitter niet heb overgenomen, dan stel ik voor dat je de designer nog even inschakelt. Vergeet dan niet aan de designer het antwoord op bovenstaande vraag door te geven.

secretarisjongedemocraten commented 6 years ago

Haha, ik kwam hierop omdat hij gekoppeld was aan het secretarismail, maar ik moet zeggen dat ik alleen secretaris had gezien en niet bashazeborg. Ach, ik kan ermee leven om als Bas door het leven te gaan. Een mogelijkheid om te reageren op de tweets zou ik nog wel mooi vinden. Verder vind ik het al heel mooi (zowel praktisch als esthetisch).

Met vriendelijke groet,

Pauline Gerth van Wijk,

Secretaris Jonge Democraten

T: +31 (0)6 16327311 E: secretaris@jongedemocraten.nl W: www.jongedemocraten.nl

Op 31 oktober 2017 om 21:05 schreef mathemaat notifications@github.com:

Interessant dat Pauline actief is onder @bashazeborg https://github.com/bashazeborg . Is dat een account dat van secretaris op secretaris wordt doorgegeven?

@bashazeborg https://github.com/bashazeborg Welke zaken mis jij nog in mijn versie van de Twitterfeed? Denk aan acties als reply, retweet en favorite, zoals ook in de huidige site.

Als je mijn versie wat saai of bleekjes vindt, wat ik me kan voorstellen aangezien ik die kekke icooontjes van Twitter niet heb overgenomen, dan stel ik voor dat je de designer nog even inschakelt. Vergeet dan niet aan de designer het antwoord op bovenstaande vraag door te geven.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jonge-democraten/website/issues/203#issuecomment-340890604, or mute the thread https://github.com/notifications/unsubscribe-auth/AJrDlneciMr0JVqzZ3eqo8zYqNjMf33Kks5sx32jgaJpZM4QB3uv .

bartromgens commented 6 years ago

@mathemaat Wat is de status van dit issue?

Lukt het om dit voor komend weekend af te krijgen?

mathemaat commented 6 years ago

@bartromgens

De code was bijna klaar, dus ik had me voorgenomen het de 9e afronden. Bij nader inzien heb ik het nu maar vast even gedaan. Pull request is onderweg.

bartromgens commented 6 years ago

PR merged dus dit issue kan dicht.