Viewall Patten breaks if a folder includes a markdown file

[Seanom]

I am using Pattern Lab Node v3.0.0-beta.1 on Mac, with Node v10.11.0, using a Vanilla Edition.

When a markdown file is added to the directories parent folder the View All link for that section breaks resulting the following error message in the console.
Uncaught TypeError: Cannot read property 'innerHTML' of null traced back to the following line patternlab-viewer.js:formatted:6047

Using the md file at the folder level appears to be the issue, removing or changing the name(so it does not match a directory) appears to resolve the issue but removes the text from the md file.

Visiting the link directly displays the page without the error message.

Selecting the View All for all of the patterns also appears to work this is limited to the folder that also has an associated md file.

The issue looks to be caused by the partial having trouble getting the div with ID pl-pattern-data-footer from the JS

document.getElementById("pl-pattern-data-footer").innerHTML,

Expected Behavior

Adding a md file to a folder displays additional information above the mustache content.

Actual Behavior

Adding a md file at the folder level causes an issue within the JS that results in Pattern lab repeating the navigation bar and displaying the Loading Pattern Lab message, until the page is refreshed.

Steps to Reproduce

1. Install the node version of PL

2. Install the basic mustache starter kit

3. Within the 00-atoms folder ensure that there is a 00-global directory

4. Within the 00-atoms folder ensure there is a global.md file

5. Build the PL and visit the local site

6. From the dropdown go to Atoms > Global > Viewall

7. Observe that the error message is displayed

   

8. Enter the URL into the browser http://127.0.0.1:3000/?p=viewall-atoms-global

9. Observe that the page displays correctly

10. Press the link on the page Global

11. Observe the page reloads and displays the broken page again
    This time the error message is slightly different
    

12. Remove or rename the global.md within the 00-atoms directory and rebuild PL

13. From the dropdown go to Atoms > Global > Viewall

14. Observe the page now loads correctly

[Seanom]

I did try changing the folder to remove the 00- from both atoms and the global folder and also adding 00- to the global.md file, but still experienced the same issue.

[bmuenzenmeyer]

This is a duplicate of #890, and matches the debugging I have done so far on the issue.

[Seanom]

Ahh sorry, I did some searching first for duplicates, but done so in the wrong repo

[stale]

It's hard to keep track of everything. This issue has been automatically marked as stale because it has not had recent activity, neither from the team nor the community. It will be closed if no further activity occurs. Please consider adding additional info, volunteering to contribute a fix for this issue, or making a further case that this is important to you, the team, and the project as a whole. Thanks!

[stale]

Issue closed after going stale. It can be re-opened if still relevant.

[stale]

Issue closed after going stale. It can be re-opened if still relevant.

[bmuenzenmeyer]

@sghoweri is this still an issue we need fixing?