-
Notifications
You must be signed in to change notification settings - Fork 6
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
Remmina Logo and Icons Redesign #3
Comments
This is not to push nor to stress, we are preparing a new release and we were wondering if we will have something to work on before that moment or not. Really no stress at all, just curious. Thanks! |
Thanks for the ping @antenore this slipped a bit behind. You will have a few logo proposals by Sunday :) |
No rush! Thanks for the reply |
@antenore what do you think about these directions? |
They're really good, but we'd would prefer logo letter working on the original R, or around that theme. |
Why didn't you specify that before? You said that you'd be open to a redesign. If this was clear in the beginning we wouldn't need to go back and forth. |
Sorry for this. Let me see with the other members, I'm not alone. Personally I prefer the second one. |
Would it be possible to have the second green and blue in SVG so that we can try to replace the original one here: https://github.com/FreeRDP/Remmina/blob/next/remmina/desktop/scalable/apps/remmina.svg Without seeing it in action we are not able to "imagine" it |
They both confirm for this one I've tested :-) |
Let me know if you want to go ahead with that. :) |
Yes absolutely. We are that kind of people that have an hard time to become habit to the changes, but now we have passed that initial obstacle, and we love it, you can go ahead. Thank you. |
Solid! Will export the logo for you then. Continuing on the icons next. |
@AnXh3L0 can you push them to the repo? |
Sure, they are now online at the repo, here. |
@antenore are you happy with the logo? Is that wrapped up? I don't see it on the website yet |
@elioqoshi yes I'm happy thank you, but it's not yet in the source code because I want to do everything in one shot with the icons. CC @giox069 |
@elioqoshi I'll be far few days, so I won't answer till Monday or Tuesday. Before to expose the new logo we really want to have the icon as well. So we will wait you. Soon there will be a new release, if you think you can do it we can wait. Than we should also discuss about the cost. Can you contact me by email? My nickname At simbiosi.org |
I will continue with the icons and get back, sounds good |
Hello @antenore, |
Feedback appreciated. Please note that the icons are color independent so that depends on the UI where they will be |
Even an 'M$' what do you think? :-D |
I have made the changes you suggested @antenore and added the icons as svg in the zip below so you can test them in the UI. |
Thanks! Today I'm not sure I'll be able to take a look, I'm out the whole day. |
Really gorgeous :-) |
@AnXh3L0 |
Thank you @AnXh3L0 ! |
So here we are... First of all, good job!! These are 2 tests with a quite default icons theme and one with an high contrast one. The spice icon doesn't look so good because I've converted it to 16x16 and 22x22 px. Is it possible to modify the original vector one so that it takes more vertical space (to have a square at end)? Maybe giving a bit of prospective? I'd like also the feedback of @giox069 , @ic3d and the other most active Remmina contributors (if the want) |
Can you try the new ones below @antenore ? I made them square so they should all look consistent. |
Thanks @AnXh3L0 I'll have a look tomorrow. |
Size are good now :-) thanks. @elioqoshi don't forget to contact me by email. ;-) |
@elioqoshi @AnXh3L0 it's strange I didn't get any email I took a look also on the server there is nothing filtered. let's try @gmail.com could be some blacklist on my or on your side. |
Hi @antenore, I resent the email to you at gmail. Below you have the spice over ssh icon. |
@elioqoshi @AnXh3L0 I'm going to update the remmina_about.glade to add your names as contributor, what do you want I write as you mail? https://github.com/FreeRDP/Remmina/blob/master/data/ui/remmina_about.glade#L158 |
Hey @antenore, you can add my email as anxhelo @ lushka dot al. Thanks! |
@antenore we would prefer to credit Ura Design in general as we think a collective of people is a stronger concept than any individual :) A credit is not required though, all the rights to the works are yours (we have to clarify the licensing from the icons which are Material Design based though) |
I'll add it.
For the license let me know.
Thanks
…On June 5, 2018 5:52:35 PM UTC, Elio Qoshi ***@***.***> wrote:
@antenore we would prefer to credit Ura Design in general as we think a
collective of people is a stronger concept than any individual :) A
credit is not required though, all the rights to the works are yours
(we have to clarify the licensing from the icons which are Material
Design based though)
|
Hi all. Sorry to come back regarding the icons, but have a little problem. I've switched the way I use the icons, setting them as symbolic icons. With the toolbar icons everything works perfectly, but with the protocol icons not. These are the rendered protocol icons and as you can see the RDP one is just a round circle. Have a look also on the other protocols icons (to me they looks good) if you think anything is missing compared to the original. The toolbar icons are perfect! Again with a dark theme I'm trying to fix it by myself but I'm afraid I'm not very competent :-P |
Ah! I've got it! I'm uploading the updated file. |
The four squares where placed above a full/filled circle (and lock in the second one). So I've reduced the alpha of the squares till 0, selected all the path and executed a difference, to have only one shape with the transparent squares. |
…n 1.2.32 Antenore Gatta (201): Addes snap build status badge String wrapping not allowed in old CMAKE versions String wrapping not allowed in old CMAKE versions Added error management Using macro GET_OBJECT Resetting macro GET_OBJECT Merge pull request #1656 from FreeRDP/builder_crash Add resize grip Merge pull request #1661 from FreeRDP/ref_main New icons preview New icons all sizes Fix toolbar icon sizes Added new icons to replace stock Updated icon name for acctions Updated pan-down symbolic icon Added remmina-ssh icon Added new icons Merge pull request #1663 from der-eismann/fix-german-typo Added Artist Anxhelo Lushka from http://Ura.Design - uradotdesign/works/issues/3 Merge pull request #1664 from FreeRDP/icons Migrating to SVG icons format Icons colour chooser dialog Adda nacl supporta tor libdsh #983 Removed statis build flag for libssh Revert lib-nacl Merge branch 'master' of github.com:FreeRDP/Remmina Revert lib-nacl Readd lib-nacl Added libffi Updated icons size Fixed colors Merge pull request #1666 from FreeRDP/icons Moving to gitlab Update README.md Adding plugin column in the Remmina Main window Merge pull request #1669 from FreeRDP/refact Moving to symbolic icons to match theme colors Updated VNC symbolic icon Fixed RDP protocol icons transparency Fixed protocol icons transparency Fixed SSH protocol icons transparency Removed colour chooser Merge pull request #1673 from FreeRDP/icons Updated issue template Merge branch 'patch-1' into 'master' Merge branch 'master' into 'master' Added SSH_AUTH_SOCK access Commented ssh-auth access as not yet released Updated crontibuting guidelines Add Printer driver and name as per #1660 and #1351 Merge branch 'rdp_printers' into 'master' Fixed ssh kex type option Merge branch 'feature/choose_correct_colorscheme' into 'master' Updated contributor lists Adding RDP Gateway RPC Transport Type Adding gitlba-ci Fixed variables Fixed variables Added apt dependencies Added image xenial Should fix apt-get issues No confirmation vte version bump Update packages after adding PPA adding flatpak build flatpak: dependencies flatpak: dependencies flatpak: add docker flatpak: add docker flatpak: moved curl call flatpak: trying distro docker package flatpak: enable docker flatpak: enable docker service flatpak: rollback docker flatpak shared submodules CI optimisations CI optimisations Allow master Removed travis and fixed cache make flatpak available for download Added flatpak repo Merge branch 'master' into issue-1687_gwrpc Set http transport to FALSE Don't use flatpak script Manually set manifest path Building as user Rolling back Trying to fix cache and artifacts Trying to fix cache and artifacts flatpak CI optimizations Various fixes Merge branch 'issue-1687_gwrpc' into 'master' Merge branch 'feature/dynamic_color_loading' into 'master' Added stdlib to avoid compiler warning Merge branch 'feature/dynamic-colors-memory-handling' into 'master' Revert "Merge branch 'feature/dynamic-colors-memory-handling' into 'master'" Merge branch 'revert-49d2ed2d' into 'master' Kiosk mode, initial implementation Force fixed size in kiosk mode Hide unneeded toolbar buttons Fullscreen in kiosk mode Kiosk mode desktop entry Kiosk mode data updates and xsession Added xsession support for Remmina Kiosk + Gnome Fixed sensitiveness of fullscreen buttons Kiosk: Code and interface cleaning Kiosk mode, initial implementation Merge branch 'feature/dynamic-colors-memory-handling' into 'master' Using g_malloc instead of malloc Merge branch 'patch-1' into 'master' Merge pull request #1680 from scootergrisen/patch-4 Merge branch 'fix_filezilla_sftp' into 'master' Merge branch 'respect_older_vte_versions' into 'master' Fixing typos Translations improvements Merge branch 'translations' into 'master' Preparing for release v1.2.31 Merge branch 'release_prep' into 'master' Revert broken translations Switching flatpak Remmina source code to gitlab Fixing snap build - removing nacl Added man entries for the kiosk mode Updating metadata Adding man pages Merge branch 'master' of gitlab.com:Remmina/Remmina Removing unneded man page Fixing snap remmina id Fixed StartupWMClass v1.2.31.1 - Bugfixing release. - Closes #1696 via ec4ad1759726aaaa08559cee1d61bae9ca57e8be - Closes #1700 via ec4ad1759726aaaa08559cee1d61bae9ca57e8be - RDP: fix to EndPaint via 16ecb49f5eb6434256342cfa676a4e2c7c3ea9fb - Various snap and flatpak fixes Merge branch 'v1.2.31.1' into 'master' Bug fixing: bumping version Bug fixing: bumping version Merge branch 'libssh_threads' into 'master' Adding error check on remmina_pref_save Implement send ctrl+alt+fn keys. Closes #1707 Merge branch 'issue-1707_spicekeys' into 'master' Improving file type hadling Preparing for release v1.2.31.3 Fixed security warning Libssh upstream hotfixes Merge branch 'ref_remmina-file' into 'master' Remove deprecated gtk_widget_reparent Fixing libssh deprecations Merge branch 'kbgrab' in to master. Merge branch 'deprecations' into st Revert "Remove deprecated gtk_widget_reparent" Revert "Remove deprecated gtk_widget_reparent" Merge branch 'deprecations' into 'master' Preparing for release v1.2.31.4 Merge branch 'rel_v1.2.31.4' Added Socket Terminal plugin Releasing Remmina Simple Terminal - #1459 Return false if terminal executable is not found Cleaning and fixes Allowing send events in xterm Minor fixes Adding vim and emacs Correcthly dealing with additional arguments in emacs and vim Merge branch 'st' into 'master' Force program name to app id Add option to honour https_proxy and http_proxy environment variable Merge branch 'app-id' into 'master' Using rfi->settings to ignore proxy settings Code cleaning Merge branch 'issue_1719-proxy' into 'master' Printing builds flags with remmina --full-version command option. Merge branch 'build_config' into 'master' PROXY_TYPE_IGNORE is available only with a recent freerdp Added function to allow x11 forwarding Merge pull request #1681 from TeknoMobil/patch-1 Merge branch 'master' into 'master' Merge branch 'patch-1' into 'master' Enable SH_AUTH_SOCK access Merge branch 'master' into 'master' Fixing TR translation Merge branch 'sshx11' Merge branch 'patch-2' into 'master' jekyll in the url is not needed as it's rewritten Adding global preference for search bar visibility - closes #1670 Merge branch 'searchbar' into 'master' Minor fix - Uninitialized variable Adding debug Adding RDP GW authentication Gateway Transport Type Screenshots custom file name Copy screenshot to clipboard Adding general option to disable screenshot clipboard Merge branch 'ref_screenshot' into 'master' Merge branch 'gw_refact' into 'master' Do not register socket plugins when X11 is not available Merge branch 'gtksocket' into 'master' BUG - GW auth data was saved in the server auth data Merge branch 'ref_gateway' into 'master' Plugin declare it uses GtkSocket and rcw block the call if not supported Removing GtkSocket test ui cleaning Avoid to save last_success property is stats are not enabled. Closes #1732 Merge branch 'issue1732_stats' into 'master' GTK_WINDOW casting Preparing for release v1.2.32 Releasing v1.2.32 Denis Ollier (20): flatpak: update lz4 from 1.8.1.2 to 1.8.2 readme: flatpak run command does not support --user arg flatpak: build libssh with libnacl support flatpak: fix libnacl build on arm tool_hello_world: add missing icon ci: remove unused MANIFEST_PATH variable flatpak: update spice-protocol from 0.12.13 to 0.12.14 flatpak: update xkbcomp from 1.4.1 to 1.4.2 flatpak: update spice-gtk from 0.34 to 0.35 remmina: improve remmina_pref_file_load_colors() function flatpak: rename DBUS_ID to REMMINA_APP_ID readme: update ci badge URL sftp: use glib's g_format_size_full() to compute file size rdp: remove unused variable remmina: use g_memdup() instead of copy in for loop remmina: fix build on CentOS 7 Fix URLs github -> gitlab flatpak: update freerdp from 2.0.0-rc2 to 2.0.0-rc3 flatpak: update libssh from 0.7.5 to 0.8.1 flatpak: update libssh from 0.8.1 to 0.8.2 Frank Niessen (12): bugfix: synchronize color scheme order of enum color_schemes and ssh_terminal_palette recap custom color prefs for vte terminal to separate struct move reading terminal color prefs to separate function get memory and fill with presettings for terminal colors 'on the fly' load terminal color from file if possible change foldername from colors to theme add using of system data dirs and sort entries in the popup menu cleanup code: switch from c-lib memory handling (malloc) to glib based memory handling bugfix: remove memory leak cleanup code: switch from c-lib memory handling (malloc) to glib based memory handling bugfix: set correct number of records for allocating memory bugfix: set correct struct parameter for foreground and background priort to vte 0.38 Giovanni Panozzo (23): Add RDP error message Merge branch 'rdp_mustchange' into 'master' Fix RDP begin/end paint which caused a disconnect/reconnect loop Merge branch 'rdp_paint_fix' into 'master' RDP sound setup RDP sound setup Fix toolbar setup after kiosk mode changes Missing man pages RDP: fix to EndPaint RDP: implements update->PlaySound for beeper. Fixes #1705 RDP: implements keyboard state update Merge branch 'rdp_leds' into 'master' RDP: support for libfreerdp complex regions in EndPaint Merge branch 'rdp_paintregions' into 'master' Set program class to REMMINA_APP_ID, fixes #1706 cmake: include libssh_threads only when available Merge branch 'readonly-stats' into 'master' Snap: update to libssh 0.8.0 Fixes a crash deleting XDMCP profile Fix some possible crashes when reading a remminafile Fix KB grabbing when switching workspace Allow wayland backend again when GTK >= 3.22.27 Merge branch 'reenable_wayland' into 'master' Jialin Wang (3): Update remmina_filezilla_sftp.sh: make it possible to use special character in password like "@" Update remmina_filezilla_sftp.sh: $password does not show in $env, use $ssh_password instead. Update remmina_filezilla_sftp.sh: 1. use "ssh_password" instead of "password" because latter does not work; 2. Use quotes to make special character like "@" in ssh password possible. Matteo F. Vescovi (1): Fix manpages format Oleg Gadelshin (1): RDP: add passwordispin option Philipp Trulson (1): Fix typo in German translation Serdar (2): Update tr.po Update tr.po Serdar Sağlam (2): Update tr.po Replace tr.po Hi Language translation of all lines c01o (2): Fix typo Fix broken links larchunix (1): Merge branch 'tool_hello_world/missing-icon' into 'master' scootergrisen (1): Add danish translation to org.remmina.Remmina.desktop.in 朝歌 (1): Update redirections in README.md
From the Open Source Design job board:
opensourcedesign/jobs#304 (comment)
The text was updated successfully, but these errors were encountered: