Latest Update Messed Up Live Search.

Posted in: Newspaper
Post count: 221

Hello, team tagDiv.

First of all, you added the functionality to disable live search on mobile & amp. Great. But it should only disable populating results in real-time and everything else should be working like before (the search slide should work the same way with animation).

Second, the search feature doesn’t work as it should be. When the icon is clicked, reloading starts, and the search page opens with every single article, every single page on the website popping up on users’ faces. When did the search start working this way? It should be freaking blank with nothing but search bar and just that, there shouldn’t be any articles and pages. You guys should’ve kept it as it was and just have provided the feature to disable live search on search slide. This search, of now, is ugly.

Third, I kept the live search enabled because I want the animation but not the live search. So I tried to disable it from the ajax_mob.php as I did previously. It doesn’t work anymore. The problem I am facing now after commenting the code is that the search in amp doesn’t work. When I type in the search bar and click go on the keyboard, nothing happens.

So please, tell me a way to keep the same search slide with animation and only disable the live search. Also, even after the removal or commenting of the code, typing in the search bar still sends POST requests to the server but doesn’t just show the results on the screen. How can I stop these requests as well?

So here is what I am looking to achieve:-

  • I want to keep the same search appearance as it was before(that slide or window that comes from the right side.)
  • I want to disable the live results showing up on both amp and mobile (achieved this by commenting the code but search on amp doesn’t work anymore.)
  • Even after commenting the code, every time a letter is typed in the search bar, a POST request is sent that consumes server resources. I want to stop that.
  • Please tell me what code to comment in which file to do what. The new search is the worst feature that I’ve ever seen in a theme update.

    Update means progress not regress.

    P.S.- All the above points and disappointments are targeted at the developers and designers and they have nothing to do with the support staff.

    Post count: 18283

    Hello !

    Please contact us via email at contact@tagdiv.com and provide the following:

    – admin url and credentials
    – cPanel url and credentials
    – A link to this topic in order to identify you

    Our team will take a look !

    Thank you !

    Post count: 221

    My guess is—you didn’t understand the issue.

    To address the mentioned issue, there’s nothing you’d be needing my website login credentials for.

    Everything’s fine with my website. It’s just a theme update that I didn’t like and want to keep old functionality that would require little tweak with the code.

    It’s a simple task. You just have to tell me which part of the code do I have to modify.

    I kindly request you to hand this issue over to Calin.

    I’ve really had a bad experience with you the last few times.

    Post count: 35449

    Hi pratiek,

    I just checked our theme code and the way it is working, therefore if you want to have the search animation but not the ajax results you need to go in wp-content/plugins/td-composer/mobile/includes/td_ajax_mob.php and comment 2 lines of code (if you comment all code will no longer work on amp) -> https://i.imgur.com/8HxqPxO.png -> https://i.imgur.com/BZMH30M.png and for amp you need to go in wp-content/plugins/td-composer/mobile/header-amp.php and comment this part of code -> https://i.imgur.com/Tzg7xDI.png -> https://i.imgur.com/awTSu40.png
    The results will be:
    amp – https://i.imgur.com/fDfB00r.png
    mobile – https://i.imgur.com/k40LGkD.png

    Hope this will help you, also I made a note to our developers with the suggestions you’ve made to can keep the search animation and remove only the ajax search!
    Let me know the results!
    Thank you!

    Post count: 221

    Nah man!

    That’s not even nearly good.

    1: I don’t want “Here are the results for the search:” text and “VIEW ALL RESULTS” button either. But they’re still there but only on the amp.

    Now you’d say that’s impossible because that button is used to view the results. Right?

    It doesn’t freakin work with Go or Search button on user’s phone keyboard. Right?
    But why? Why am I not able to search articles pressing the Search or Go button on my phone’s keyboard?

    What if I want to delete that ugly “VIEW ALL RESULTS” button? And I want that deleted.

    2: Another problem that you ignored perhaps due to ignorance. This time I will write that with emphasis so you get that completely into your head:—

    WHEN. WE. TYPE. A. LETTER. IN. SEARCH. BAR. IT. SENDS. AJAX. REQUEST(GET). TO. THE. SERVER. TO. GET. RESULTS.

    The above problem not only happens when we type but also when we change cursor position(only mobile not amp).
    I checked that on desktop with inspect elements using arrow keys.

    This is a waste of server resources and put extra load on it. I don’t want any ajax results to show, so it should not send any request.

    Problem 1 is only with mobile theme. Problem 2 is with both amp and mobile theme.

    Here’s a screenshot for text and button, and for the ajax requests.

    Also Note: In mobile theme every time we type or change cursor position, request is sent but on amp theme if you type 5 letters quickly at once it will only send 1 request and 5 if type them slowly with a difference of 1 second or perhaps even less.
    So if after typing 5 letters rapidl, if you type one more letter it will send one more request totalling 2.

    The change in cursor position sends request only on mobile not amp. To test this, open inspect element and render mobile theme. Open networks tab in inspect window(it could be different for macOS), you will see when you type in search bar a request is sent and when you change cursor with arrow keys, another request is sent. That’s a lot of complaints.. Hope you have a solution.

    Please show this to developers and tell them to buy a good course from Coursera. Udacity is also good.

    Have a nice day,
    You do a great job.
    Pratiek.

    Post count: 405

    @Pratiek: Same problem!!! And worst is we don’t know if it will work in the next update if we fix the issue. With every update, we have to fix something.

    Post count: 35449

    Hi pratiek and Nagarig,

    Please note that in this way was design the live search, that’s how it should work, now it was add the option to remove the live search, but you still do not like the way its displaying now the mobile search. What I’m trying to do is to help you to find a solution. Your ask is for a custom work these are perform by developers, not support staff, but I ‘m trying to help with a little custom work that I’m able do do.

    To remove completely the ajax from mobile search you need to replace the js from file wp-content/plugins/td-composer/mobile/js/tagdiv_theme.min.js with this code -> https://pastebin.com/LBJxbytw in this way the mobile theme will no longer call the td_ajax_mob.php unfortunately for amp I can not find a way to prevent the ajax to be use.

    Thank you for your understanding!

    Post count: 221

    Take it easy, bruh. Take a deep breath.
    I am not asking you to find the code which is responsible for this. But, you can’t say that this is how it should work, as you said in the first line because you don’t know anything about UI and UX. So better dry up.

    Also,
    That is not an ask for the custom work. I’ll explain why.
    The developers should educate the support staff about the basic stuff. If someone says that they want to disable the live search, they only want to stop the requests being sent to the server to prevent overuse but still want to have the animation and everything else as it was earlier. Simple. That is an easy and obvious assumption to be considered by developers. So they should teach the support staff about things like that.

    But developers tried to solve this problem by giving users a feature to disable live search from the panel. But while solving a problem, they destroyed the user experience and compromised the feel and convenience. Now, this is not support-staff’s fault. Everybody knows this already. But what support staff can do instead of getting mad at customers is to give feedback to developers so that they can provide a solution or at least know that what they did is not much liked.

    And if support staff is not allowed to ask for help from developers, then how can they help customers because they know nothing about coding and design.

    If you believe that this is how search should work, then please see the Salient theme. It also has a non-ajax search, but the user doesn’t have to leave the page to type in the search box. Why shouldn’t it work like this? I know it’s not your job. But this post should be shown to the developers because they’re trying to get away and avoiding the tough work. Because removing the ajax feature while still keeping the animation would’ve required a lot of code modifications in the javascript that they don’t want to go through. So they just did a little copy-pasting, which was comparatively pretty simple.

    I love this theme. And that’s why it’s upsetting me that developers didn’t think about customer’s expectations. When you work so hard to make your customers fall in love with your company and your work, you avoid doing stupid things that can make them hate you.
    Either do it well or don’t do it at all.

    Post count: 405

    I agree with him. @Calin: We just want a simple search with sliding animation and no ajax requests. There are so many sites with same functionality, so you should consider this in the next update.

    Post count: 18283

    Hello !

    We are sorry for the misunderstanding !

    You are correct and thank you for working with Newspaper Theme.

    I have passed this issue directly to our developers and they will have a look at it. Once a decision is being taken we will come back to you.

    Thank you for your understanding !

    Post count: 405

    Hello, after 2 theme updates, this issue is not solved. We still have to face this Mobile Search problem.

    Post count: 18283

    Hello !

    Please contact us via email at contact@tagdiv.com and provide the following:

    – admin url and credentials
    – A link to this topic in order to identify you

    Our team will take a look !

    Thank you !

    Post count: 405

    @Vlad: Do you read what we are saying or just randomly copy-paste that ADMIN URL answer to everyone? 🙁

    @Calin
    , Hope you will answer about this.

    Post count: 35449

    Hi,

    This option with a normal search on mobile theme plugin is still on our list, for the moment these updates were made only to fix the Instagram widget and for WordPress 5.5 compatibility, there are no new elements added. Also I spoke with a developer and he said that they are working on mobile theme plugin to make it better, unfortunately I do not know what new options will be add or when this will be released, but it should be in the coming updates.
    Thank you for your understanding!

    Viewing 14 posts - 1 through 14 (of 14 total)
    The forum ‘Newspaper’ is closed to new topics and replies.