Hey Alex,
This was something that had hopefully been fixed in a previous version of Reboot.
Are you all up to date? If so could you supply a link?
Ike said
Hey Alex,This was something that had hopefully been fixed in a previous version of Reboot.
Are you all up to date? If so could you supply a link?
Yes, all up to date. I’m using regular Reboot for mobile, and child for desktop. Just created a test post here: http://www.ohhtee.com/forum/guests/video-test/
That’s a bit odd. I’ve checked that link in FF, Chrome and on my phone and all I’m getting is Reboot Desktop no matter what viewport size or user agent on desktop..
kvr28 said
this is what I am seeing on a galaxy s5
Sometimes they will show, though most get cutoff when on mobile. I use an Iphone. Is your browser set for mobile? Asking because it shows the side where stats are.
Yes, set for mobile. Weird, I just went back to double check and reboot showed fine
Ike, the flex wrap seems to still be an issue with ios, I just double checked it with my ipad and see the same, I had noticed it after the update as well but forget to mention it on the reboot thread I had made.
Thanks Kyle, I’m having no luck recreating this locally! Unfortunately I don’t have any Apple devices to hand..
I too am now getting mobile theme, but it’s only mobile theme, no desktop.. There’s definitely something strange going on there!
Alex, could you try adding some CSS to test? If you’re happy to, please open reboot-mobile.php. Under the very first media rule could you make some space and add:
#spMainContainer .spTopicPostSection .spPostSection,
#spMainContainer .spPostContentSection {
display: inline;
}
The start of the file should then look like:
@media screen and (max-width: 2000px) {
#spMainContainer .spTopicPostSection .spPostSection,
#spMainContainer .spPostContentSection {
display: inline;
}
And let us know if it resolves the wrap issue?
So I added the code:
@media screen and (max-width: 2000px) {
#spMainContainer .spTopicPostSection .spPostSection,
#spMainContainer .spPostContentSection {
display: inline;
}
#spMainContainer .spGroupForumSection.spOdd,
#spMainContainer .spForumTopicSection.spOdd,
#spMainContainer .spMemberListSection.spOdd {
color: <?php echo($indexRowOddColor); ?>;
background: <?php echo($indexRowOddBackGround); ?>;
border: <?php echo($indexRowOddBorder); ?>;
}
Whereas before it was like this:
@media screen and (max-width: 2000px) {
#spMainContainer .spGroupForumSection.spOdd,
#spMainContainer .spForumTopicSection.spOdd,
#spMainContainer .spMemberListSection.spOdd {
color: <?php echo($indexRowOddColor); ?>;
background: <?php echo($indexRowOddBackGround); ?>;
border: <?php echo($indexRowOddBorder); ?>;
}
Now the video is showing up normally, though the overall formatting has some issues:
Before:
OK, good stuff – I can at least see the problem in Safari now!
You’ve got the code in the right place, so could you now try changing the
display: inline;
to
display: block;
Ike said
OK, good stuff – I can at least see the problem in Safari now!You’ve got the code in the right place, so could you now try changing the
<br />display: inline;<br />to
<br />display: block;<br />
Awesome! That did it! Will this be fixed, or is this something I should add the to the child?
It would probably be better for you to add the change to Reboot itself rather than your child theme.
The next version of Reboot will contain the fix so when that’s released and you update, your version will then contain the fix and your child theme can remain just changing the elements you want it to change, not fixing things!
Leaving it out the child theme and putting it in the parent also means that if we find a better way of fixing it up than the method above, you don’t then have to remove this method and replace it with the new one.
Awesome. going to try this later on a theme I have been wanting to change to. Thanks ike!
No problem, let me know if there are any knock on effects, I wont be able to properly test it until I meet with Andy (Who has Ipad, Iphone).