So useful – so simple

  • I just installed this plugin a few minutes ago. It’s already configured and working perfectly.

    I’m giving a 5-star rating because it does exactly what it promises. That said, I would like to suggest two important areas of improvement.

    My goal when installing this plugin was to ensure my main menu navigation stuck to the top of the screen when scrolling. That was immediately resolved after installing the plugin and providing the class name of the menu. Unfortunately, this plugin supports only a single class, and when my site is displayed on a mobile device, the class name changes.

    I’m suggesting that the class name be tied to the screen size. This plugin allows for functionality to be enabled/disabled based on screen size. It would be more useful to also include the ability to generate a different class name based on screen size.

    Also, a suggestion to help users with less HTML skills… when prompting for the class name, it might be useful to include information in the “?” popup on HOW to find the class name (e.g. using firebug or viewing element information while browsing the website).

    • This topic was modified 8 years, 2 months ago by jefromcanada.
Viewing 3 replies - 1 through 3 (of 3 total)

  • Thanks for your review, much appreciated! ??

    Unfortunately, this plugin supports only a single class, and when my site is displayed on a mobile device, the class name changes.

    To be honest, I’ve never seen a theme do it that way actually (swapping a classname of a menu or a header between desktop and mobile). In your case, I’m sure there is an element name that you can target that will be the same for both mobile and desktop. For example, if the element would be something like this on desktop:

    <div class="header">
    <div class="my-desktop-menu"> ..... </div>
</div>

    And this on mobile:

    <div class="header">
    <div class="my-mobile-menu"> ..... </div>
</div>

    Then you could target it in both cases by entering something like “.header div” — that way it wouldn’t depend on the changing class names.

    Maybe it’s a bit more complex in your case, but I hope you see what I mean.

    Having said that, when I will add the option to make multiple elements sticky, you could do it the way you want to. Have “.my-mobile-menu” sticky on small screens only, and have “.my-desktop-menu” sticky on large screens only. This update has been on my radar for a long time, but I still have no ETA for that…

    As for your second suggestion: I have thought about this, but it’s not easy to write instructions that are simple and concise. For example, different users use different browsers (so the documentation would have to apply to all possible browsers), another reason is that finding a class name is not an “exact science”. Sometimes you want to target the menu, sometimes you want the parent element of the menu (for example, because that is where the background color is applied), sometimes the grandparent, etc. etc. It all depends on how the site is styled, and there are so many possibilities available.

    Hence, describing all possible ways of how and where you can locate the classname would almost be a whole book! Or, well, it would be a lot of documentation… So this is why it’s basically expected that the user knows their way there. Of course, if they can’t find it or have no idea how to do it, I will always be happy to help when someone posts a message on the support forum and I can take a look at it. ??

    Once again, thanks for your feedback!

    • This reply was modified 8 years, 2 months ago by Senff - a11n.
    Thread Starter jefromcanada

    (@jefromcanada)

    Mark,

    I appreciate you replying to my review. I had noticed you make an effort to reply to NEGATIVE reviews (a good policy by the way), but assumed you wouldn’t feel the need to reply to a positive review. So, thanks for that.

    When I mentioned needing more documentation for how to find class information, I didn’t intend it to mean that YOU need to supply that documentation. Only that you may choose to mention that most browsers have a means of displaying that information, and leave it to the user to take it from there.

    As far as finding a common element on full-sized and mobile sites, I was able to find a common element (the header). Unfortunately, keeping the full header from scrolling on a mobile site leaves precious little room left to display anything else!

    Hi Jeff,

    As long as the review itself has a question or suggestion that I’d like to address, I’ll reply — regardless of being positive or negative. ??

    Gotcha on the documentation! It’s actually a good idea to point to third party documentation for finding the proper HTML element. I’ll have to find a good one for that.

    If you need any further technical help or suggestions on your header issue, I’d be happy to give it a look; just post a message in the support forum and I’ll check it out!

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘So useful – so simple’ is closed to new replies.