holderdeord / hdo-site

Parliamentary monitoring website for Norway
https://data.holderdeord.no/
BSD 3-Clause "New" or "Revised" License
42 stars 16 forks source link

Add FB app #661

Closed Andersos closed 10 years ago

Andersos commented 10 years ago

Disse må legges til i FB appen om de skal fortsette å ha tilgang: www.facebook.com/635985218 www.facebook.com/1562377886 www.facebook.com/518536137 www.facebook.com/525020960 www.facebook.com/757795452

Kan du gjøre det @jarib

Skal fikse logo til appen og gjøre den aktiv så merger jeg om det er ok.

jarib commented 10 years ago

Jeg får ikke lagt til @tiinar og @linnskorge fordi de ikke har verifisert kontoen sin. Guro og Daniel er ikke lenger aktive og kan fjernes.

Andersos commented 10 years ago

Vet vi om de aktivt følger med på Facebook statistikk? Er det ok at vi krever at de verifiserer kontoen for å få ny tilgang til dette?

jarib commented 10 years ago

Har lagt til Tiina og Linn som «Insight users» - det krever ikke verifisering og burde gjøre susen.

jarib commented 10 years ago

Ser ut som Open Graph Object Debugger er fornøyd!

jarib commented 10 years ago

Dette ligger nå i produksjon, men ser ut som Facebook ikke lenger skjønner at vi er en norsk side?

F.eks. på sakssiden

image

og forsiden:

image

Ser også ut som Facebook-knappen har fått noe padding-top el.l. som den ikke hadde da vi la den til opprinnelig (men tror ikke det er et resultat av fb-app-endringen).

Kanskje du kan ta en titt ved anledning, @Andersos ?

Andersos commented 10 years ago

Jeg finner ut av det. Paddingen på FB knappen så jeg tidligere. Skal prøve å finne ut av den også.

jarib commented 10 years ago

Eller kanskje er det Tweet-knappen som trenger mer padding?

image

http://next.holderdeord.no/issues/262-etablere-flere-grunnskolelaererutdanninger-for-realfag

henrikhelmers commented 10 years ago

iframe#twitter-widget-0.twitter-share-button.twitter-tweet-button.twitter-count-horizontal { margin-bottom: -7px; } Gjør noe, men er ikke nødvendigvis sunt. Supplér med margin-top: 10px; for mer luft i toppen. Men det er ikke på linje med aside under tittelen.