Skip to content

Documented Twig Component attribute #2793

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: 5.0
Choose a base branch
from
Open

Documented Twig Component attribute #2793

wants to merge 6 commits into from

Conversation

mnocon
Copy link
Contributor

@mnocon mnocon commented Jun 17, 2025 •
edited
Loading

@mnocon mnocon force-pushed the twig-component-attributes branch from d4fa21c to 8e56520 Compare June 17, 2025 08:08
@github-actions GitHub Actions
Copy link

Preview of modified files

Preview of modified Markdown:

adriendupuis
adriendupuis reviewed Jun 18, 2025
View reviewed changes
mnocon and others added 4 commits June 18, 2025 12:09
@mnocon @adriendupuis
Update docs/templating/components.md
89ca2a4 
Co-authored-by: Adrien Dupuis <61695653+adriendupuis@users.noreply.github.com>
@mnocon
Fixes
324dcc6
@mnocon
Make the examples the same
88c08d6
@mnocon
Added highlight
675e242
@mnocon mnocon requested review from wiewiurdp and adriendupuis July 3, 2025 11:55
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 3, 2025

code_samples/ change report

Before (on target branch)After (in current PR)

code_samples/back_office/components/MyComponent.php


code_samples/back_office/components/MyComponent.php

docs/templating/components.md@26:    ``` php hl_lines="8-11"
docs/templating/components.md@27:    [[= include_file('code_samples/back_office/components/MyComponent.php', glue='    ') =]]
docs/templating/components.md@28:    ```

The following line doesn't include file correctly:     [[= include_file('code_samples/back_office/components/MyComponent.php', glue='    ') =]]


Download colorized diff
      		
    

  





        


            

            
            

              
            

        

      


      

            
  
  

    
  
    
    

  

  

  


    
















  
  

    

      

  




      


    


    

      

        
      

  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment


  



      

    

  




  
          




      

  

    
    

    

  

    Reviewers
  


    


      

          


  
      @wiewiurdp
  
    wiewiurdp


                Awaiting requested review from wiewiurdp


        
      

      

          


  
      @adriendupuis
  
    adriendupuis


                Awaiting requested review from adriendupuis


        
      


      






    

  


      
  

    Assignees
  



        


    No one assigned







      


  


  

    Labels
  



    

      

    Needs DEV review









      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    

      
        

          
    

    Development
  



            


Successfully merging this pull request may close these issues.



  
  
  



      
    

  




      

    

  
      

  

    

      2 participants
    

    

        
          @mnocon 
        
          @adriendupuis