Fix admin bar site status HTML content

[wpalani]

No description provided.

[circlecube]

I think we wanted the content to contain both states, so toggling the state would be much easier. The visibility of each state is controlled via CSS and the attribute data-coming-soon. This section is the associated styles that should control visibility: https://github.com/newfold-labs/wp-module-coming-soon/blob/main/includes/ComingSoon.php#L150-L158

Rather than only including the current content, I think just the test should be updated to check that the proper span is visible in the admin bar. #nfd-site-status-live and #nfd-site-status-coming-soon. That will make it less reliant on text content too (for translations and any other content customizations - since the text can be overridden from the plugin).

[wpalani]

I think we wanted the content to contain both states, so toggling the state would be much easier. The visibility of each state is controlled via CSS and the attribute data-coming-soon. This section is the associated styles that should control visibility: https://github.com/newfold-labs/wp-module-coming-soon/blob/main/includes/ComingSoon.php#L150-L158

Rather than only including the current content, I think just the test should be updated to check that the proper span is visible in the admin bar. #nfd-site-status-live and #nfd-site-status-coming-soon. That will make it less reliant on text content too (for translations and any other content customizations - since the text can be overridden from the plugin).

Roger that! I'm closing this PR and will tackle the tests update in a different PR along-side other changes.