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

! Some updates to the calendar template and css #529

Merged
merged 2 commits into from Jun 17, 2013
Merged

! Some updates to the calendar template and css #529

merged 2 commits into from Jun 17, 2013

Conversation

Spuds
Copy link
Contributor

@Spuds Spuds commented Jun 15, 2013

Signed-off-by: Spuds spuds@spudsdesign.com

Antechinus added 2 commits June 15, 2013 07:49
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
@StealthWombat
Copy link
Contributor

BTW, if you throw this lot in, it shouldn't break anything else. The calendar stuff is self-contained in the CSS. This one should be totally safe to run live (hint hint).

norv added a commit that referenced this pull request Jun 17, 2013
! Some updates to the calendar template and css
@norv norv merged commit 8d27ea1 into elkarte:master Jun 17, 2013
@norv
Copy link
Contributor

norv commented Jun 17, 2013

This one should be totally safe to run live (hint hint).

It's live. :)

@StealthWombat
Copy link
Contributor

Aha. Missed one little bit here. Had it on local but either Spuds or I missed it with the PR.

Down in the pile of crud (slated for execution soon):

/* The generic wrapper thingy for teh awesome cool looks innit. */ /* This lot will be simplified once I get all the styling hooks in place. */ .generic_list_wrapper, .action_profile #permissions, #postmodify .roundframe, .add_buddy, #creator .windowbg2, #notify_options .windowbg2, #info_center, #profileview, #messageindex, #unread, #mlist, #main_grid table, .core_posts, #recent .core_posts, #recent .topic_table, #searchform .roundframe {

Get rid of this:

#main_grid table,

That wll fix the last little bit. :)

@Spuds
Copy link
Contributor Author

Spuds commented Jun 17, 2013

I'm not going to make a PR for that (since this one is closed and merged now anyway) ... Just have it as part of the next big update and it will get in ... till then a fat border on the calendar :P

@StealthWombat
Copy link
Contributor

Fair nuff. :D Just thought I'd mention it.

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

Successfully merging this pull request may close these issues.

None yet

3 participants