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

The calendar is not displayed correctly #19

Open
SebasAcevedo opened this issue May 20, 2015 · 6 comments
Open

The calendar is not displayed correctly #19

SebasAcevedo opened this issue May 20, 2015 · 6 comments

Comments

@SebasAcevedo
Copy link

I just downloaded the plug in and run the index.html and all the calendars show wrong
eventcalendar

@jaime8111
Copy link
Owner

Looks like a CSS problem.

With out having access to the web page I can't help you. sorry.

2015-05-20 18:15 GMT+02:00 Sebastián Acevedo Flórez <
[email protected]>:

I just downloaded the plug in and run the index.html and all the calendars
show wrong
[image: eventcalendar]
https://cloud.githubusercontent.com/assets/4970630/7730703/6f2b3ac8-fee1-11e4-91eb-423935b25d83.PNG


Reply to this email directly or view it on GitHub
#19.

Jaime Fernández
· [email protected]
· Twitter http://twitter.com/vissit · Facebook
http://www.facebook.com/jaime8111 · Linkedin

http://www.linkedin.com/in/jaimefdez

@clews
Copy link

clews commented May 29, 2015

Had the same problem. I did not really take note of all the css classnames I changed, but some of them missed the eventscalendar- prefix and some had eventcalendar without the "s"... Perhaps this hint helps...

@mechamyu
Copy link

Hello SebasAcevedo, Jaime,

I've had this issue a couple times now when downloading the newest version of the code, too.

However I was able to find a fix, thanks to Clews' hint. Indeed, many CSS declarations in the CSS Core and the Responsive theme do not have the .eventsCalendar- prefix, and in the core, the issue is mainly that all prefixes are actually not plural (eventCalendar rather than eventsCalendar). However, the .eventCalendar-wrap class must be kept singular, as the JavaScript still renders this one (all other classes must have the plural prefix.)

I don't know yet how to help on code/github stuff so I sure hope this information helps correcting the CSS for users in the future. I would attach the corrected core CSS and Theme if I could, but I fear copy/pasting it here might be spammy..

All in all, in both these files, no classnames should be left unprefixed. so if you see .empty... it should be .eventsCalendar-empty.

So far the only exceptions I found were the classes .today and .bt
Cheers,

@jaime8111
Copy link
Owner

Thanks for the info.

As you said It is a problem with my last update. I hope to fix it tomorrow

El jueves, 16 de julio de 2015, Karine Frenette-Gaudreault <
[email protected]> escribió:

Hello SebasAcevedo, Jaime,

I've had this issue a couple times now when downloading the newest version
of the code, too.

However I was able to find a fix. Many CSS declarations in the CSS Core
and the Responsive theme do not have the .eventsCalendar- prefix, and in
the core, the issue is mainly that all prefixes are actually not plural
(eventCalendar rather than eventsCalendar). However, the
.eventCalendar-wrap class must be kept singular, as the JavaScript still
renders this one (all other classes must have the plural prefix.)

I don't know yet how to help on code/github stuff so I sure hope this
information helps correcting the CSS for users in the future.

Cheers,


Reply to this email directly or view it on GitHub
#19 (comment)
.

Jaime Fernández
· [email protected]
· Twitter http://twitter.com/vissit · Facebook
http://www.facebook.com/jaime8111 · Linkedin

http://www.linkedin.com/in/jaimefdez

@SebasAcevedo
Copy link
Author

Thanks all for the support

@jaime8111
Copy link
Owner

It is now fixed.

Thanks again for your message

2015-07-17 0:09 GMT+02:00 Sebastián Acevedo Flórez <[email protected]

:

Thanks all for the support


Reply to this email directly or view it on GitHub
#19 (comment)
.

Jaime Fernández
· [email protected]
· Twitter http://twitter.com/vissit · Facebook
http://www.facebook.com/jaime8111 · Linkedin

http://www.linkedin.com/in/jaimefdez

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants