Modern theme style elements issues

[isigma]

Here is a list of elements that does not have a proper look in the modern theme, or bugs. (WPLMS 1.9.9.1, Modern 1.1)

1. Course Category widget does not excludes terms (mod_101.jpg)
2. Course category terms separator (comma) on the course archive page appears at the beginning of each consequent line: before the term, but not after ( mod_102.jpg). Also is there a possibility to display terms inline?
3. When MyCred price for a course exists, the price on the archive page is being displayed as if the price drop-down list is already opened with improper width and extra line below.  (mod_103.jpg) The drop-down box works properly  on the course details page though.
4. Confusing course duration format: on the archive page it's in Days (I prefer that), on the course details page it's in old approximation format (which works aweful, e.g. 35 days is being displayed as "month", not even "1 month", which is still not equal 35 days) (mod_104.jpg). How to change course duration format to Days on course pages?
5. I do not understand the design of the course price area on the modern course style page. The price is being displayed both under the instructor on the sidebar, and in the header. It looks ok if we have the only price, but still confusing, as the sidebar doesnot have a "Take course" button, where people would expect it near the price. Now when we have MyCred price, it's being displayed as a drop-down box, so we can change the price independently in both areas (say, side bar is in Creds, but header is in $$), still having only one "Take course" button on the page. Isn't it confusing to see different price on the same page? Which one is the valid onr for the button?

[isigma]

PART 2 6. Languages us_EN.po/mo files either needs to be updated or deleted, as they contain phrases like: "M Reviews" for "More Reviews", and "R Courses" for "Related courses" 7. CSS problem with the hover "More Reviews" bar. It doesn't reach left edge all the way. (mod_106.jpg). You can check it on your demo site as well. 8. Problem with Related Courses module. It displays the parent course as well (e.g. THIS course is related to itself). <span style="line-height: 1.5;">You can check it on your demo site</span><span style="line-height: 1.5;"> </span>http://themes.vibethemes.com/wplms/skins/modern/course/software-training/ 9. After changing the Related Courses style in the WPLMS Course settings for the first time, I can not change it again. All options display the same (post style) style. 10. in Firefox Bubbypress Profile images (comments, course authors) are empty. == #4 I found the answer to question 4. It's in the WPLMS Course settings.

[Anshuman Sahu]

1. yes this is an issue, thanks for reporting this to us .We will get it fixed in upcoming update of wplms . 2. Could not replicate the issue .please check : http://prntscr.com/92kfd6 3. Yes this is an issue with the dropdown as of now with the modern theme we will get it fixed also with the next udpate of modern child theme . 4. Please note that if you are logged in and have already took the course then on the single course page it will show the remaining time for you in the course details while it ill show the actual course duration that was set in the all courses page . So these two time will be different .Also if the duration is set to automatic in wplms -> course manager ->  Course Duration parameter ,then it will adjust according to the duration left for you in the course . Like if duration initially set was 30 days the it will show a month after one day it will began to show 29 days and will follow to hours then minutes and so on . i.e the duration changed from month to days . 5. WE just used the course details widget in the sidebar from the default theme in the modern theme .It includes the pricing also and we have also added the pricing above the button in the modern theme . Therefore it is appearing two times in that page . You can remove the price form the course details by adding the given code in your wplms-customizer.php file in wplms customizer plugin . add_filter('wplms_course_details_widget','custom_wplms_course_details_widget'); function custom_wplms_course_details_widget($course_details){  $key = 'price'; // Set $key value from price,precourse,time,level,seats,badge,certificate unset($course_details[$key]); return $course_details; } 6. Yes this is also an issue with the translation files of the modern theme ,you can edit these translations using the code styling plugin . refer :  https://wplms.io/support/knowledge-base/translate-using-codestyling-localization-plugin/ 7. Try adding this css for this : #buddypress .course_reviews .show_course_reviews .show_reviews {     margin-left: -20px;   } 8. Yes this is also an issue with the related courses widget . We will get it fixed in the next update of vibe course module plugin . 9. This is also an issue wea re trying to fix this and will be fixed in the upcoming update of wplms modern theme . 10 . COuld not replicate this issue please check :  http://prntscr.com/92knbv

[isigma]

2. Please look at your own screenshot: each category line starts with comma, not ends with. 5. This is good, but the header's price doesn't have a dropdown, so it's not possible to choose MyCreds units there. 10. please check the attachment Firefox 42.0 Windows Thank you for other solutions and great support! Will be waiting for the update.

[Anshuman Sahu]

2. :) try adding the given css from customize : #buddypress #course-list .modern_course_single_item .course-category li { width: auto; clear: none; color: #BBB;   } 5. Yes the same dropdown will appear that appear in the course details widget and will have mycred points . 10. Checked on firefox 42 on windows : http://prntscr.com/93b80b Please share site url to check this at your end . /  

[isigma]

10. It may be an issue with my local server. I'll check it when the project is online. Thank you

[isigma]

As I mention in my 5-star review, that your efforts to upgrade great functionality is beyond praising. There is only one question to the theme - it's design. It's not only about the style, but about little details and absence of empty space. Just one default p {line-height:1.5em} makes it breeze. There are also small bugs here and there. I know how difficult and time consuming is to support a theme with wide functionality, and really would like to help you to take care about those little things. So far my style.css for modern Child is 10kB. Some of the lines in it is my personal restyling of the theme, but number of them are fixes. Would you like me to send you specific to the original theme stylesheet with corrections (and description) if it'll make your life easier? Like (not in stylesheet), that the thumb_modern.png is missing in the assets/images folder of the modern theme, which results in default missing image logo in the page grid builder module. With high respect, Igor

[Anshuman Sahu]

Tnhaking you for your concern modern theme is the child theme of the wplms theme and we are fixing the modern theme in each and every update . You can share your style.css file here if you want to submit correction with the styling of modern theme .

[isigma]

Here you go. I tried to exclude my restyling, but left some thoughts together with bugs corrections. Hope it will save you some time. /* Theme Name: WPLMS Modern Version: 1.2 */ /*/////////////////// - HEADER AND MENU - ///////////////////*/ #headertop li {border-left-color: #e9e9e9 !important} /* top menu dividers has wrong color */ .icon-search-2:before { /* search icon in main menu is not aligned and size might be corrected according to typography settings*/ font-size:15px; margin-left:10px; } .directory .pagetitle { /* course header image */ padding: 50px 0 65px; /* numbers may differ, but posts and courses originally has different title bar height */ } more-title-area .more-title-content .title-text, .more-title-area .title-content .title-text, .title-area .more-title-content .title-text, .title-area .title-content .title-text { padding: 60px 0 50px; /* post header image */ } /*///////////////// - TYPOGRAPHY - /////////////////*/   select, input {font-size:14px} /* select forms: originally text doesn't fit into the field*/ /* Tables - up to you, but I'd recommend to add need some table css, at least td,th {    padding:10px; }*/ p {line-height:1.5} /* not a bug, but makes the theme breeze. Up to you. To support it I also increased  line-height for h1-h6*/ /*/////////// - POSTS STYLE - //////////*/ /* There is no consistency in page design in terms on margins for the .container. Here is how I made them looking closer on different type of page templates */ .container {padding:20px 0 0} #headertop .container {padding:0} @media (max-width: 991px) { .container { padding: 0; } } @media (max-width: 767px) { .container { padding: 0 20px; } }       /*//////////// - COURSE STYLE (buddypress) - ////////////*/ #buddypress .course_reviews .show_course_reviews .show_reviews {margin-left: -20px;} /* bug in More Reviews button border  */ #buddypress #course-list .modern_course_single_item .course-category li+li:before {color:#bbb} /* meta devider had a different color from the meta text */ #buddypress .course_header div#item-header .course_price, #buddypress .group_header div#item-header .course_price, #buddypress .member_header div#item-header .course_price {color:white !important} /* header arrow in the price dropdown had a different color */ /*////////// - BBPRESS FORUM - ////////////*/ legend { width: auto; border-bottom: 0px; } /* I'm not sure if the border was by mistake, but with BBPRESS it looks better withoout */ @media screen and (max-width: 768px) { /* vertical tabs have to go horizontal in mobile design, otherwise tabs and content are way too separated in a single column */ .tabs .nav-tabs > li {width: auto !important} .content .tabs-right .tab-content, .content .tabs-left .tab-content {width:100%} }  

[Anshuman Sahu]

Thanks for sharing the css fixes.

[Author]

Posts