Jump to content

Community

Inconsistent theme HTML


ehren.

Recommended Posts

Always been that way unfortunately.

Appears they do not have a standard set of rules they make their coders use, seems they are left to it and do as the coders want to do and not worry about if they are missing code that other coders use in their designs.

Pointed this out to IPS way back to v2 and always been there every release, might fix one part in following release but then miss another part so gave up trying to tell them.

Makes it damn awkward for us themers but until someone tells them to use a basic standard set of rules then will always happen (was hoping this was going to happen when v4 was released as being re-written from scratch)

Not much fun if trying to make theme to cover all IPS apps when there is coding missed in some of the apps and have to try and make it work without touching templates

Another thing is when they change the css used for areas to something else (what is wrong with using the old css for those areas) why change it.

In my opinion seems some things are just not thought through properly before work starts

Link to comment
Share on other sites

4 hours ago, Mark said:

That looks like a bug (I actually had the particular case you mention in my notes as I'd noticed it myself too). If you can provide a list of pages you've seen this on, I'll make sure they get fixed.

Thanks Mark - I'm sure a tonne of my customers will appreciate that.

Pretty much every edit form needs styling. The below screenshot is from the Blogs, but the same happens with the Gallery and Download addons.

5981c85cbdbe7_ScreenShot2017-08-02at10_40_05pm.png.5cf7c0c14006dec39dbd7be8f182aeeb.png

 

There are numerous areas that contain page titles or descriptions. This example is from the Store (when viewing a category of products). The ipsType_medium class is also used in the section title for some reason, which makes the text excessively small.

5981c9fa77349_ScreenShot2017-08-02at10_46_30pm.png.894bb4dd443c6cd13c7bfea886933f92.png

 

The album description when viewing albums (below the Rules box if it's not clear in the screenshot):

5981cae28e375_ScreenShot2017-08-02at10_50_55pm.png.8ed099e66a1d8beaa1702491ae61b2bb.png

 

If the rules are displayed on the page (instead of a popup box or link):

5981cb9a0f950_ScreenShot2017-08-02at10_54_25pm.png.fcd55134901eab1e8da7fa10656b2b3b.png

 

The ModCP:

5981cb0f45c71_ScreenShot2017-08-02at10_52_15pm.png.9ff44312548130390726487f41e4b764.png

 

There are numerous pages where the large page title sits outside the content:

5981cd8bed6d6_ScreenShot2017-08-02at11_02_37pm.png.1f130ff524519ad8e8031ac2a7456555.png

 

The inbox, when no message is selected.

5981cdbda91b3_ScreenShot2017-08-02at11_03_24pm.png.0e8cc40689c57029901183ee1bdd071b.png

I think customers have also reported issues with the Terms and Conditions and privacy - although I could be wrong.

I'm sure there are many more areas, but those were found after a quick sweep through my forum. An easy way to find these issues is to simply change the page background colour to a dark colour and you'll notice how easily these areas stick out.

If the "text on transparent background" effect is intended, then please just add a new class to the parent of those elements (such as .ipsContent_transparent or something) which will allow developers to add backgrounds to those areas if necessary.

Link to comment
Share on other sites

@Mark - What's totally disappointing to me is that in 4.2, you can't use the killer built in theme editor with a dark background because most of the text will also be dark and you simply can't change that in certain areas.   In some cases, the text needs to be dark and in others, light, However, there are not options to change the text color.

Can this be fixed by adding more ways to change the font colors in the theme editor under text?

I really love the theme editor and hope this can be fixed.

Edited by jackflash
Link to comment
Share on other sites

On 8/5/2017 at 9:53 PM, liquidfractal said:

I also notice this problem with some manually-coded Pages pages, such as my in-progress About page, below:

In this instance, you should add the wrapper manually - wrapping all Pages that use the suite wrapper can actually result in undesired effects for some clients.

Link to comment
Share on other sites

I guess this is as good a thread as any to also point out:

For those of us who use @Kevin Carwile's Collabs app, there are also many pages in that app that do not have the proper wrappers for certain areas of content.  I'll list some when I have time, but for now I want to point out that this is a significant CSS issue with Collabs' integration with the core suite as well.

Link to comment
Share on other sites

On ‎08‎/‎08‎/‎2017 at 7:18 AM, liquidfractal said:

I guess this is as good a thread as any to also point out:

For those of us who use @Kevin Carwile's Collabs app, there are also many pages in that app that do not have the proper wrappers for certain areas of content.  I'll list some when I have time, but for now I want to point out that this is a significant CSS issue with Collabs' integration with the core suite as well.

That you would take up with the author (it may be author hasn't spotted it and if no one inform them then how can they fix the issue)

Link to comment
Share on other sites

In our guide for upgrading themes to 4.2 there is a section about this change called "No content background color by default" - does adding a background as specified there (which will take you back to a similar approach as used for 4.1) help? As you've noticed, some of these areas are headers which are deliberately not inside ipsBox elements.

 

Link to comment
Share on other sites

4 hours ago, Mark said:

In our guide for upgrading themes to 4.2 there is a section about this change called "No content background color by default" - does adding a background as specified there (which will take you back to a similar approach as used for 4.1) help?

The issue existed in 4.1 and likely 4.0 too, it's not unique to 4.2. The guide which you linked to isn't related to the issue's we've mentioned, so adding a background colour to the entire site wrapper isn't the solution we're after unfortunately.

We don't need the entire #ipsLayout_contentWrapper area styled, we need the individual content areas correctly wrapped with ipsBox. The HTML structure is correct on most pages, but other pages are broken (shown in all of the screenshots above) since they lack any form of structure. If you need to reproduce this on your own site, simply change the Page Background theme setting to a dark colour such as #3c3c3c and visit any of the affected pages. It'll become quite obvious :)

Additionally, the headers which are deliberately outside of the ipsBox elements should at least have a class wrapped around them which can give themers the opportunity to easily style them.

Link to comment
Share on other sites

  • 2 weeks later...
  • 2 weeks later...
On 8/19/2017 at 2:01 PM, MiP1 said:

IPS Staff, any news on this? Hope it can be fixed with the next update.

I'd also really like to know if there will be movement on this issue in the future.  These gaps look unsightly when one is trying to market professional services via a website, and as much as I like the default Blue theme some of my users want more choice and eyecandy as well.

Link to comment
Share on other sites

Another area that needs to be addressed is the notValidated template.  Currently it gives you this

 

Screen-Shot-2017-08-28-at-9_58.24-PM.jpg.98dbfd4c2ffbdabc330798f5b1670d05.jpg

 

it wouldn't be so bad if there was an effective template history/merge capability, but after each style upgrade you have to remember to redo these special template edits since apparently when you upgrade your style you should install it fresh and not "upgrade" it.  You then have to redo any special template edits that you have created.  It's really a pain and one of the great weaknesses that I've found in the IPS templating system.

Edited by Tracy Perry
Link to comment
Share on other sites

  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...

Important Information

We use technologies, such as cookies, to customise content and advertising, to provide social media features and to analyse traffic to the site. We also share information about your use of our site with our trusted social media, advertising and analytics partners. See more about cookies and our Privacy Policy