MagicMirror Forum
    • Recent
    • Tags
    • Unsolved
    • Solved
    • MagicMirror² Repository
    • Documentation
    • 3rd-Party-Modules
    • Donate
    • Discord
    • Register
    • Login
    A New Chapter for MagicMirror: The Community Takes the Lead
    Read the statement by Michael Teeuw here.

    BBC News feed

    Scheduled Pinned Locked Moved Utilities
    10 Posts 3 Posters 6.9k Views 3 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • T Offline
      TamP
      last edited by

      Yes isnt it. I try this and it didnt seem to do anything
      Its not a big deal but I just wonder if I had done something wrong

      1 Reply Last reply Reply Quote 0
      • justjim1220J Offline
        justjim1220 Module Developer
        last edited by

        there is an omission in the newsfeed.js file, that is causing this problem…

        this secton:

        if(!this.config.showFullArticle){
                var title = document.createElement("div");
        	title.className = "bright medium light" + (!this.config.wrapTitle ? " no-wrap" : "");
        	title.innerHTML = this.newsItems[this.activeItem].title;
        	wrapper.appendChild(title);
        }
        

        states if(!this.config.showFullArticle){

        But there isn’t a showFullArticle: false; in 'defaults

        So, basically it is ignoring the showSourceTitle: true, because this section cannot be executed

        I think it would make more sense to have the above code snippet be worded like this:

        if(this.config.showSourceTitle){
                var title = document.createElement("div");
        	title.className = "bright medium light" + (!this.config.wrapTitle ? " no-wrap" : "");
        	title.innerHTML = this.newsItems[this.activeItem].title;
        	wrapper.appendChild(title);
        }
        

        then maybe add:

        showSourceTitle = false;
        

        to this section of code:

        if (this.config.showFullArticle) {
        	var fullArticle = document.createElement("iframe");
        	fullArticle.className = "";
        	fullArticle.style.width = "100vw";
        	// very large height value to allow scrolling
        	fullArticle.height = "3000";
        	fullArticle.style.height = "3000";
        	fullArticle.style.top = "0";
        	fullArticle.style.left = "0";
        	fullArticle.style.border = "none";
        	fullArticle.src = this.getActiveItemURL()
        	fullArticle.style.zIndex = 1;
        	wrapper.appendChild(fullArticle);
        }
        

        if you don’t want the title to display while the article is being displayed.

        "Life's Too Short To Dance With Ugly People"
        Jim Hallock - 1995

        T 1 Reply Last reply Reply Quote 1
        • T Offline
          TamP @justjim1220
          last edited by

          @justjim1220 I tried altering the code as you suggest but unfortunately I am still not seeing the BBC title

          justjim1220J 1 Reply Last reply Reply Quote 0
          • T Offline
            TamP
            last edited by

            This post is deleted!
            1 Reply Last reply Reply Quote 0
            • justjim1220J Offline
              justjim1220 Module Developer @TamP
              last edited by

              @TamP

              you might try deleting the module then reinstalling it

              I just tried exactly what you have configured and it works just fine…

              Since newsfeed is a part of the initial MM2 installation, go here: https://github.com/MichMich/MagicMirror/tree/master/modules/default/newsfeed
              and get the newsfeed files themselves. You probably only need the actual js files: fetcher.js & newsfeed.js
              right click on the file and click Save Link As… then save to your newsfeed directory within MM2

              "Life's Too Short To Dance With Ugly People"
              Jim Hallock - 1995

              T 1 Reply Last reply Reply Quote 1
              • T Offline
                TamP @justjim1220
                last edited by

                @justjim1220 Yes that is what I did and all is OK
                Thanks again

                1 Reply Last reply Reply Quote 1
                • 1 / 1
                • First post
                  Last post
                Enjoying MagicMirror? Please consider a donation!
                MagicMirror created by Michael Teeuw.
                Forum managed by Sam, technical setup by Karsten.
                This forum is using NodeBB as its core | Contributors
                Contact | Privacy Policy