Skip to content

Logo (Builtin): Add Kylin Logo (Remove ANSI codes)#1760

Merged
CarterLi merged 3 commits into
fastfetch-cli:devfrom
zx900930:distro-kylin
May 16, 2025
Merged

Logo (Builtin): Add Kylin Logo (Remove ANSI codes)#1760
CarterLi merged 3 commits into
fastfetch-cli:devfrom
zx900930:distro-kylin

Conversation

@zx900930
Copy link
Copy Markdown
Contributor

@zx900930 zx900930 commented May 16, 2025

Fixed ASCII Logo, Remove all the escaped ANSI codes

                 ++                     
             *  ***                     
           *******                      
         *******++                      
      ********+===                      
    ******#*+======                     
   *******#***++===                     
   *********+**===                      
  ##*********##==                  ====+
%##****++==+****#                ***+=**
#****++=====++****##            ********
####*+=======+******===-     ***#*##****
 #%#******+**+==++**=======+*+*#%       
 %#****+******=====+====+++==**         
  #*********##***###********##%         
   ***##%#%%%%*****#*###*****#%         
   *#%%%#%%%%%****   %%*******#%        
   #%%%%%%          %%%##***###%%       
    **#%           %%%%#% **#**##       
   ###%%         %%%%%   %%%***         
 %%%%%%        %%%%%%%  %%%%#*          
 %%%%%%                %%%%%%+=        

This is a fix for #1756

@zx900930 zx900930 mentioned this pull request May 15, 2025
@CarterLi
Copy link
Copy Markdown
Member

CarterLi commented May 16, 2025

You must add color placeholders in your ASCII logo, or the colors you set in src/logo/builtin.c won't work.

Take ubuntu_kylin.txt as an example.

@zx900930
Copy link
Copy Markdown
Contributor Author

@CarterLi Thx for your patience. I have adjusted my code to add the proper color prefix according to my builtin.c snippet.

See 5c61676

here is the preview
image

@CarterLi CarterLi merged commit 2d93804 into fastfetch-cli:dev May 16, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants