Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Twitter Module Stuck in "Loading" State #26

Closed
Area51Brett opened this issue Feb 5, 2014 · 10 comments
Closed

Twitter Module Stuck in "Loading" State #26

Area51Brett opened this issue Feb 5, 2014 · 10 comments
Labels

Comments

@Area51Brett
Copy link
Member

As we all have seen the Twitter module on the bottom of the page is broken and stuck saying "Loading". It looks to be an issue with the Twitter API, we should either fix it or remove it from the site because currently it looks unprofessional.

@notslang
Copy link
Contributor

notslang commented Feb 5, 2014

See issue here: core2062/website#1
Closing because this is a duplicate and this project is going to be deprecated by core2062/website soon. Not even sure if it's worth releasing a new version of this to patch it.

@notslang notslang closed this as completed Feb 5, 2014
@notslang notslang mentioned this issue Feb 8, 2014
@nightpool
Copy link
Contributor

Reopened. core2062/website is no longer on our short term radar, and this needs to be fixed.

@nightpool nightpool reopened this Feb 8, 2014
@nightpool nightpool added the bug label Feb 8, 2014
@agilgenbach
Copy link
Collaborator

Workaround discussed at:

seaofclouds/tweet#264

@agilgenbach
Copy link
Collaborator

Clicked closed by accident.

@nightpool
Copy link
Contributor

Workaround not necessary. Haley said she'd be working on this tonight at
the meeting.

Unlike tweet.js, we're supported fine by Twitter's official API.

On Tue, Feb 11, 2014 at 2:51 PM, agilgenbach [email protected]:

Clicked closed by accident.

Reply to this email directly or view it on GitHubhttps://github.com//issues/26#issuecomment-34804676
.

@notslang
Copy link
Contributor

except the official twitter widget doesn't fit as well into our site, and doesn't seem to offer styling.

@Featherdragon15
Copy link
Member

The official widget does in fact offer some customization (like simple
things such as border color and link color) but honestly the dark theme
fits with our existing website just fine.

On Tue, Feb 11, 2014 at 3:17 PM, Sean Lang [email protected] wrote:

except the official twitter widget doesn't fit as well into our site, and
doesn't seem to offer styling.

Reply to this email directly or view it on GitHubhttps://github.com//issues/26#issuecomment-34808034
.

Haley Bates

#################################################################################################################################################################################################################################

@notslang
Copy link
Contributor

This dark theme?
image

...That matches our theme even less than the default widget. It doesn't match the background color of the other content-boxes, uses rounded borders (unlike everything else on the page), doesn't use flat-buttons, has seemingly random margins (we use a standard measure of 10px nearly everywhere), and the font-sizes / fonts are different (twitter uses Helvetica Neue and we use Calibri).

But perhaps the worst part is that it introduces a scroll-bar just for the widget. I mean, how selfish is that?! Jeez, Twitter - you're just a widget, not the center-piece of the page! You don't deserve to have your own scrollable content-section; at most you get a "filler" space at the bottom of the page where you can be casually looked upon.

@Featherdragon15
Copy link
Member

Yes, okay, thank you Sean, but honestly a working widget that doesn't fit
exactly with the theme is better than one that perpetually loads and never
displays tweets.

On Tue, Feb 11, 2014 at 8:07 PM, Sean Lang [email protected] wrote:

This dark theme?
[image: image]https://f.cloud.github.com/assets/1049204/2144447/51782afc-9387-11e3-8c0f-c6f35f52fe3e.png

...That matches our theme even less than the default widget. It doesn't
match the background color of the other content-boxes, uses rounded borders
(unlike everything else on the page), doesn't use flat-buttons, has
seemingly random margins (we use a standard measure of 10px nearly
everywhere), and the font-sizes / fonts are different (twitter uses
Helvetica Neue and we use Calibri).

But perhaps the worst part is that it introduces a scroll-bar just for the
widget. I mean, how selfish is that?! Jeez, Twitter - you're just a widget,
not the center-piece of the page! You don't deserve to have your own
scrollable content-section; at most you get a "filler" space at the
bottom of the page where you can be casually looked upon.

Reply to this email directly or view it on GitHubhttps://github.com//issues/26#issuecomment-34831709
.

Haley Bates

#################################################################################################################################################################################################################################

@notslang
Copy link
Contributor

Or we could just remove the widget entirely... after all, we only tweet like 2 times a month.

notslang referenced this issue Feb 22, 2014
See StanScates/Tweet.js-Mod for more information
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants