OPNsense
  • Home
  • Help
  • Search
  • Login
  • Register

  • OPNsense Forum »
  • English Forums »
  • Development and Code Review (Moderator: fabian) »
  • Cicada Theme -> Franco didnt want the cicada theme - so deleted!
« previous next »
  • Print
Pages: 1 ... 3 4 [5]

Author Topic: Cicada Theme -> Franco didnt want the cicada theme - so deleted!  (Read 21792 times)

mimugmail

  • Hero Member
  • *****
  • Posts: 6405
  • Karma: 446
    • View Profile
Re: Cicada Theme -> Franco didnt want the cicada theme - so deleted!
« Reply #60 on: April 03, 2018, 09:17:07 am »
Quote from: noname12123 on April 03, 2018, 08:47:04 am
Quote from: marjohn56 on April 01, 2018, 03:50:01 pm
Just a note to everyone, I have received an apology from noname123123 and I hope so has @franco, so lets give the guy some slack now. We all have lessons to learn in life and none of us are perfect, well most of you aren't  :P

I would ask the mod to delete this thread after a few days so we can close this chapter and move on. I'll be taking the Cicada theme under my wings and helping to get it pushed through.

Thank you again for your welcome. I just want to say that anyone who wants to improve the cicada theme can do it. color technology can be there a lot better. I'm also working on a new theme that is just the opposite of dark. will soon put a few screenshots of the new template then online here. thanks again for your support.

If you push Cicada again this will be the same old problems. Please consider in improving the existing dark theme. You'll collect some experiences how Github works and how to work in a team. It can and will be disappointing at some stage, but everyone contributing code in a project has to learn to live with it (so as I).

For a new theme there's no worries to create a new pull request, but please, do your first run with some small PR's to an existing theme. I can't count how many github mails I received from openend and closed PR's ;)

I invite you to join the IRC channel and I can give you a short introduction to Github (also in German).
Logged
Twitter: mimu_muc
WWW: www.routerperformance.net
Support plans: https://www.max-it.de/en/it-services/opnsense/
Commercial Plugins (German): https://opnsense.max-it.de/

marjohn56

  • Hero Member
  • *****
  • Posts: 1679
  • Karma: 171
    • View Profile
Re: Cicada Theme -> Franco didnt want the cicada theme - so deleted!
« Reply #61 on: April 03, 2018, 10:13:59 am »
Quote from: mimugmail on April 03, 2018, 09:17:07 am
If you push Cicada again this will be the same old problems. Please consider in improving the existing dark theme. You'll collect some experiences how Github works and how to work in a team. It can and will be disappointing at some stage, but everyone contributing code in a project has to learn to live with it (so as I).

For a new theme there's no worries to create a new pull request, but please, do your first run with some small PR's to an existing theme. I can't count how many github mails I received from openend and closed PR's ;)

I invite you to join the IRC channel and I can give you a short introduction to Github (also in German).

I have taken this under my wing and have committed it this morning after testing it for a couple of days. There are sufficient differences between the two themes to have both. Yes, they are both dark, but Cicada uses different fonts and uses different colours in many places. I personally prefer Rebellion, others prefer Cicada.

For those wishing to try it, use the following method to install from the commit....once accepted.

# opnsense-code plugins
# cd /usr/plugins/misc/theme-cicada
# make upgrade
« Last Edit: April 03, 2018, 10:37:26 am by marjohn56 »
Logged
OPNsense 21.7 - Qotom Q355G4 - ISP - Community Fibre 1Gbps.

Team Rebellion Member - If we've helped you remember to applaud

opnsenseuser

  • Sr. Member
  • ****
  • Posts: 392
  • Karma: 25
    • View Profile
Re: Cicada Theme -> Franco didnt want the cicada theme - so deleted!
« Reply #62 on: April 03, 2018, 07:54:42 pm »
Quote from: marjohn56 on April 03, 2018, 10:13:59 am
Quote from: mimugmail on April 03, 2018, 09:17:07 am
If you push Cicada again this will be the same old problems. Please consider in improving the existing dark theme. You'll collect some experiences how Github works and how to work in a team. It can and will be disappointing at some stage, but everyone contributing code in a project has to learn to live with it (so as I).

For a new theme there's no worries to create a new pull request, but please, do your first run with some small PR's to an existing theme. I can't count how many github mails I received from openend and closed PR's ;)

I invite you to join the IRC channel and I can give you a short introduction to Github (also in German).

I have taken this under my wing and have committed it this morning after testing it for a couple of days. There are sufficient differences between the two themes to have both. Yes, they are both dark, but Cicada uses different fonts and uses different colours in many places. I personally prefer Rebellion, others prefer Cicada.

For those wishing to try it, use the following method to install from the commit....once accepted.

# opnsense-code plugins
# cd /usr/plugins/misc/theme-cicada
# make upgrade

thx very much!

maybe there are a few things we can take over from the cicada theme to the dark theme!
Logged
Supermicro A2SDi-4C-HLN4F
Team Rebellion Member (sidebar / themes: tukan, cicada & vicuna)

marjohn56

  • Hero Member
  • *****
  • Posts: 1679
  • Karma: 171
    • View Profile
Re: Cicada Theme -> Franco didnt want the cicada theme - so deleted!
« Reply #63 on: April 03, 2018, 07:57:41 pm »
Let's leave them separate. As I said, they are sufficiently different to both exist. I prefer Rebellion, but it's personal taste.
Logged
OPNsense 21.7 - Qotom Q355G4 - ISP - Community Fibre 1Gbps.

Team Rebellion Member - If we've helped you remember to applaud

franco

  • Administrator
  • Hero Member
  • *****
  • Posts: 14343
  • Karma: 1243
    • View Profile
Re: Cicada Theme -> Franco didnt want the cicada theme - so deleted!
« Reply #64 on: April 04, 2018, 08:48:49 am »
Thanks to everyone. For me the matter is resolved as well and we can move on.

I've asked noname12123 to take a little time to help shape the core theme issues as well as building the perfect theme in the plugins. The former helps everyone, helps to see how we communicate and review and improve so that the process for larger changes is more smooth for everyone involved.

Special thanks to Martin for helping noname12123 with the theme plugin inclusion.


Cheers,
Franco
Logged

  • Print
Pages: 1 ... 3 4 [5]
« previous next »
  • OPNsense Forum »
  • English Forums »
  • Development and Code Review (Moderator: fabian) »
  • Cicada Theme -> Franco didnt want the cicada theme - so deleted!
 

OPNsense is an OSS project © Deciso B.V. 2015 - 2023 All rights reserved
  • SMF 2.0.19 | SMF © 2021, Simple Machines
    Privacy Policy
    | XHTML | RSS | WAP2