SethClydesdale / forumactif-edge

A free modern and responsive forum theme for Forumotion forums.
https://sethclydesdale.github.io/forumactif-edge/
GNU General Public License v3.0
10 stars 17 forks source link

Classified Ads : Design Issues #11

Closed SethClydesdale closed 7 years ago

SethClydesdale commented 7 years ago

Topic on forum : http://fmdesign.forumotion.com/t1085-bug-some-bugs-about-the-classified-ads-on-fae#24465

Description : There's a few inconsistencies with the design of the classified ads section on forumotion. Mostly some minor tweaks will need to be made to the CSS and theme switching JavaScript.

Todo :

  1. ✔️ The "Posts" text has no purpose on the classified ads forums. Find some way to remove this text. (image)
  2. ✔️ Theme switcher has no effect on the classified ads section. Add the new selectors to the theme switcher. (click to view JS)
  3. ✔️ The texts for the ad container titles are not vertically aligned. Make some some adjustments to align the texts. (image) (click to view CSS)
SethClydesdale commented 7 years ago

Numbers 1 & 3 have been fixed and will be hotfixed later today. The fix for number 2 will be served via patch 1.1.4.

Changes made :

  1. Removed "Posts" texts under stats for classified ad forums. (live)
  2. Updated CSS selectors in the theme switcher so that it affects the classified ad forums. (Coming in v1.1.4)
  3. Fixed alignment of various texts for classified ads. (live)

I'll contact the topic author to confirm if 1 & 3 have been fixed before closing this issue.

SethClydesdale commented 7 years ago

Everything appears to be in working order and there have been zero reports since this fix, so I'll consider this issue fixed. I plan on investigating the mobile issues next, once that has been remedied I'll send out the patch.