[docs] Double-check updates to all component docs #2395

Activity

1. markcaron

   added

   featureNew feature or requestNew feature or request

   priority: lowSeverity level: 3Severity level: 3

   on Feb 28, 2023
2. markcaron

   added this to the PatternFly Elements 2.0 (aka PF 1:1) milestone on Feb 28, 2023
3. markcaron

   assigned

   zeroedin

   on Feb 28, 2023
4. kylebuch8 commented on Feb 28, 2023

   

   kylebuch8

   on Feb 28, 2023

   Contributor

   More actions

   I don't think we want to link to the demo pages. Those are intended for the developers to review every possible use case of the component. I think we want to have multiple examples on the docs page and keep the demo page unlinked.

   👍React with 👍1Reacted by Mark Caron
5. bennypowers

   modified the milestones: PatternFly Elements 2.0 (aka PF 1:1), PatternFly Elements 2.x on Mar 10, 2023
6. zeroedin commented on Mar 10, 2023

   

   zeroedin

   on Mar 10, 2023

   Collaborator

   More actions

   @kylebuch8 So currently we only show the demo link overlay when NOT in prod.

   patternfly-elements/docs/components/index.html

   Lines 24 to 33 in a57ce83

   	{%- if env.ELEVENTY_ENV == "prod" -%}
   	<a href="{{ ('/components/'+ element.slug +'/') | url }}" aria-label="{{ element.title }}">
   	<div class="preview-image" style="background-image: url({{ ('/components/'+ element.slug +'/docs/screenshot.png') | url }});"></div>
   	</a>
   	{%- else -%}
   	<div class="preview-image" style="background-image: url({{ ('/components/'+ element.slug +'/docs/screenshot.png') | url }});"></div>
   	<div class="overlay">
   	<a class="cta secondary wind" href="{{ ('/components/'+ element.slug +'/demo/') | url }}">Demo</a>
   	</div>
   	{%- endif -%}

   Otherwise, I think we can close this issue as I think this was covered in: #2398
7. zeroedin

   closed this as completedon Mar 24, 2023