[Theme] Storm Cloud

[Theme] Storm Cloud Started by Spuds · June 02, 2015, 09:49:41 pm · Read 42275 times 0 Members and 2 Guests are viewing this topic. previous topic - next topic

Re: [Theme] Storm Cloud

Reply #15 – June 08, 2016, 07:42:57 pm

It may be coming from theme.js .... if you are using Pale Moon it (when I did this at least) did not support css "cover" properly ... so I had to jack it up with some JS

Re: [Theme] Storm Cloud

Reply #16 – June 08, 2016, 10:47:28 pm

Ok. So why not just make the anchor roundy and give it hidden overflow? Then just set image width to 100px and height to auto. Should work.

Re: [Theme] Storm Cloud

Reply #17 – June 09, 2016, 03:40:50 am

Ooh dear. I made the mistake of looking at this one on an iPhone sized-screen. It gets more than a tad messy. Given that I never browse on mobile anyway, you may be reaching the limits of my enthusiasm for debugging there. If I'm going to have to put serious work in, I might as well roll my own themes.

Re: [Theme] Storm Cloud

Reply #18 – June 09, 2016, 04:04:54 am

Just making the anchor roundy and setting hidden overflow works. No need for object-fit or javascript. CSS just needs this:

Code: [Select]

.poster_avatar .linklevel1 {
	overflow: hidden;
	display: block;
	height: 100px;
	width: 100px;
	margin: 0 auto;
	border-radius: 50%;
}

With the template being:

Code: [Select]

							<li class="listlevel1 poster_avatar">
								<a class="linklevel1" href="' . $scripturl . '?action=profile;u=' . $message['member']['id'] . '">
									' . $message['member']['avatar']['image'] . '
								</a>
							</li>';

Also, syntax error here. You forgot the background-image lead in for the linear gradient, so that declaration gets dropped and in turn the one after it also gets dropped.

Code: [Select]

/* Hover effects for those buttons. */
.linklevel1.active:hover, .listlevel1:hover .linklevel1.active,
.linklevel1:hover .pm_indicator {
	background: #DDE0E3; linear-gradient(to bottom, #DDE0E3, #EFF0F3);
	box-shadow: 1px 1px 1px rgba(0, 0, 0, 0.3) inset;
	color: #555;
}

Should be:

Code: [Select]

/* Hover effects for those buttons. */
.linklevel1.active:hover, .listlevel1:hover .linklevel1.active,
.linklevel1:hover .pm_indicator {
	background: #DDE0E3;
	background-image: linear-gradient(to bottom, #DDE0E3, #EFF0F3);
	box-shadow: 1px 1px 1px rgba(0, 0, 0, 0.3) inset;
	color: #555;
}

Re: [Theme] Storm Cloud

Reply #19 – June 09, 2016, 04:29:21 am

Oh and I'm getting two undefined errors in the log.

Code: [Select]

Type of error: Undefined
8: Undefined index: image
http://127.0.0.1/Elk_latest/index.php?
File: K:/VertrigoServ/www/Elk_latest/themes/elk_theme_storm_cloud/GenericBoards.template.php (boardindex_outer_below sub template - eval?)
Line: 195
		
Type of error: Undefined
8: Undefined index: 0
http://127.0.0.1/Elk_latest/index.php?
File: K:/VertrigoServ/www/Elk_latest/themes/elk_theme_storm_cloud/GenericBoards.template.php (boardindex_outer_below sub template - eval?)
Line: 189

I haven't touched that template yet. Not me. Someone else dunnit.

Re: [Theme] Storm Cloud

Reply #20 – June 09, 2016, 10:40:29 am

QuoteJust making the anchor roundy and setting hidden overflow works. No need for object-fit or javascript. CSS just needs this:

I'll take a look at that ... for some reason I ended up using object fit but don't recall why, and then the JS was needed to cover yet another Pale Moon issue.

It may have been small or non square avatars that caused me to use object-ft, but I really don't recall anymore.

I'll check on the template error (you are using 1.0.7 yes?)

Re: [Theme] Storm Cloud

Reply #21 – June 09, 2016, 05:09:46 pm

Yup, 1.07.

Re: [Theme] Storm Cloud

Reply #22 – June 09, 2016, 10:43:37 pm

Yay! Another bug!

You've set the attachment blocks to a width of 24%. The problem is that it means clicking on the image thumbnail still leaves it as a maximum of 24% wide. IOW, about the same size as the thumbnail was anyway.

Haven't done a fix for this one yet, but it's not hard. While grumbling about this I checked the default theme too. That's had a change which borks the image scaling in attachments, so they now overflow the post div instead of fitting within it. Will also devise a fix for that, if I'm still feeling magnanimous.

Antechinus will be starting to use grumpy emoticons soon.

Re: [Theme] Storm Cloud

Reply #23 – June 12, 2016, 07:28:57 am

Yeah, I was going to mention this issue but figured you would enjoy finding it for yerself

The quick fix is to use the fancybox addon, it works great with that markup, and I know in your loins you are a fan of lightbox's !

I think whats here is mostly the 1.1 markup which was done around the simple lightbox in place of the "expand in place and mess up the page layout" expand in place version we had in 1.0 ... which of course does no good since this is a 1.0 theme. I'll take a shot at incorporating the changes you suggested in the other thread to fix this and allow the old expand in place js to work.

Re: [Theme] Storm Cloud

Reply #24 – June 12, 2016, 04:45:41 pm

I love finding them if other people love fixing them.

Re: [Theme] Storm Cloud

Reply #25 – August 23, 2019, 08:35:12 am

Hi Spuds, are you also working on an update of this theme for EA 1.1 ?

Re: [Theme] Storm Cloud

Reply #26 – August 23, 2019, 08:40:51 am

It is ... I'll work on it this weekend

Re: [Theme] Storm Cloud

Reply #27 – September 02, 2019, 05:29:17 pm

I made some updates so this works with 1.1, get that version at the theme site (link is in the first post of this thread). I made some updates for the mobile stuff, but as indicated that is one area that still needs work. Anyway this is a good start

Re: [Theme] Storm Cloud

Reply #28 – September 03, 2019, 08:51:08 am

Great, thank you! I will download it immediately.

Re: [Theme] Storm Cloud

Reply #29 – September 23, 2019, 02:34:59 am

I think there is a little bug here

(or at least I see it with Firefox 69)