stevedlawrence commented on code in PR #1434:
URL: https://github.com/apache/daffodil-vscode/pull/1434#discussion_r2392416036


##########
README.md:
##########
@@ -99,7 +99,7 @@ This product includes the 
[space-grotesk](https://github.com/floriankarsten/spac
 
 # Legal
 
-Apache, Apache Feather Logo, Apache Daffodil, Daffodil, and the Apache 
Daffodil logo are trademarks of The Apache Software Foundation.  Visual Studio 
Code and VS Code are trademarks of Microsoft® Corporation. All rights reserved.
+Apache, Apache Oak Leaf Logo, Apache Daffodil, Daffodil, and the Apache 
Daffodil logo are trademarks of The Apache Software Foundation.  Visual Studio 
Code and VS Code are trademarks of Microsoft® Corporation. All rights reserved.

Review Comment:
   The trademark blurb on the apache.org homepage just calls it the "Apache 
logo". To be consistent with that, I'd suggest we drop "Oak Leaf"



##########
README.md:
##########
@@ -70,7 +70,7 @@ If additional help or guidance on using Apache Daffodil™, 
Apache Daffodil™ E
 
 # Community Feedback
 
-[![image 5](images/asf_logo_url.png)](https://www.apache.org)
+<img src="images/asf_logo_wide.png" width="581" alt="Apache Software 
Foundation Logo">

Review Comment:
   A width of 581 seems pretty big. Is it worth making it smaller? No strong 
preference either way.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to