-
-
Notifications
You must be signed in to change notification settings - Fork 187
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.
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
Branding proposition + d-wid logo/bootsplash #1506
Conversation
Hi, I don't actually expect the xcf (from which all the jpges were sourced, with some layer manipulation) to be merged into master. The idea is that that PR serves as a repository of what I come up with, the xcf is there so that people can build on it (in the exact same way I did on the original from @Thrilleratplay ), or change the donation QR code etc without applying the JPEG compression again. Same idea with the svg. If you want to put it somewhere on the readme, website etc you can, and in any size you want, but it's obviously not a splash image (though I've just noticed the svg is pretty negligible in size compared to the xcf).
Is the plan to merge all the jpegs? In that case I think @ThePlexus gets the credit for the box if I remember correctly.
What do you mean by properly? In any case @Thrilleratplay probably gets credit for all the images since they're all just new layers on top of some of his existing ones (though it's not always the case anymore that any of his layers remain visible, not sure how that works but probably not something I ought to stress about too much?). The box versions were copied from @ThePlexus , and the Heads logo used (and the colour scheme in the tricolour version) gets credited to Microsoft of all people lol. |
b0e8eec
to
c60a254
Compare
I think I addressed all comments but squashing my commit and #1502 on previous commit. @ThePlexus @Thrilleratplay @d-wid Please look at branding/Heads/CREDITS and let me know if something is wrong. |
c60a254
to
8c4c425
Compare
8c4c425
to
fd74c4e
Compare
Nicer then I thought. @d-wid we are now co-authors on your last commit here after squashing your commit with mine. |
fd74c4e
to
18a4399
Compare
Thanks! I think this is a pretty reasonable approach. It's nice to tie the brand name and bootsplash together since they are related. |
@d-wid @Thrilleratplay @ThePlexus : a quick review on CREDITS file please? Otherwise I will merge probably by the end of the week! |
It would be more correct to say that the svg (instead any of the jpegs into which the svg was imported) was the result of Inkscape automatically smoothing out a the Bing image (after I made it all black with gimp). Bing itself didn't spit out anything in vector form after all.
Otherwise I think it looks alright.
…On 4 October 2023 21:12:40 UTC, tlaurion ***@***.***> wrote:
@d-wid @Thrilleratplay @ThePlexus : a quick review on CREDITS file please?
--
Reply to this email directly or view it on GitHub:
#1506 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
|
Any direct change you want to see @d-wid in the CREDITS file? That would be helpful! Otherwise I'll try to represent your suggestion as best as I can, but this is why we love diff/git :) |
Something along the lines of:
bing-d-wid_logo.svg: Heads Logo generated by Bing's AI on August 03 2023 with prompt "Double Headed Janus Logo for Heads Open Source Firmware" shared under #1402 (comment) *edited with GIMP and traced with Inkscape*
bing-d-wid_heads-red_blue_background-plain-3_DonateQrCode.jpg: @d-wid remix of Bing's Heads logo + DonateQrCode
|
@tlaurion I appreciate offering credit for the trivial bit of a background I threw together by sending one of the coreboot source files (I can't remember which) into a binary encoder and pasting into GIMP, but feel it is unwarranted. Had I not been so busy the past few days, I would have said not to bother mentioning me. Oh well. Thank you. These look awesome. |
18a4399
to
aaf8698
Compare
@d-wid sorry for delay. Just implemented requested changes which reflects directly your comments at #1506 (comment) Will merge in ~2 hours. This is benign change after all. |
Yeah I think so.
I meant for the "*" characters to result in italics in the above comment instead of appearing in the txt file and the last three pictures might also need "+DonateQrCode" added to them but those are minor points not worth getting boddeg down over.
…On 6 October 2023 18:19:25 UTC, tlaurion ***@***.***> wrote:
@d-wid sorry for delay.
aaf8698 looks fine?
--
Reply to this email directly or view it on GitHub:
#1506 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
|
Squash of linuxboot#1502 + moving logo/bootsplash files under branding/Heads - Move logos and bootsplashes from blobs to branding/Heads/ - Makefile: add support for BRAND_DIR which depends on BRAND_NAME which defaults to Heads if no branding - Boards coreboot configs: change bootsplash directory to depend on BRAND_DIR (instead of BLOBS_DIR) in bootsplash enabled configs - Branding/Heads/bootsplash-1024x768.jpg points to branding/Heads/d-wid-ThePlexus_coreboot-linuxboot-heads_background-plain_DonateQrCode.jpg - xcf file deleted. Original still under linuxboot#1502 to reuse for modification without recompressing (blobs/heads.xcf) - CREDITS file created to point to original authors, remixers (Open for details) - Thanks to: @d-wid for remixing Bing's AI generated Janus logo, @ThePlexus for Qubes Box concept and @Thrilleratplay for its matrix background
aaf8698
to
2bfa173
Compare
This PR is an attempt to fix #1468 by adapting #1502 from @d-wid amazing work.
@JonathonHall-Purism would like to hear from you on Makefile change and branding/Heads directory and qemu config used as example to use the default brand_name.
@d-wid : this PR works, but as stated inside of the commit message here:
I would then take your work, and modify this PR accordingy to have all coreboot configs point to the bootsplash as depicted by this PR and modify Makefile following remarks here and either merge this PR and or yours to have proper branding with logo/bootsplashes produced.
@d-wid sorry for those back and forths!