Skip to content
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

Theme-Cicada Release 1.0 #632

Closed
wants to merge 4 commits into from
Closed

Theme-Cicada Release 1.0 #632

wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 26, 2018

with sass and less adaptation

Install and screenshots:
https://forum.opnsense.org/index.php?topic=7524.0

@ghost ghost changed the title Theme-Cicada 1.0 Theme-Cicada Release 1.0 Mar 26, 2018
@fichtner
Copy link
Member

@opnsenseuser12123 for the second time, you are intertwining with theme-rebellion plugin for this sudden push to version 1.0. To be clear, all plugins start as 0.1 and, after merge and community feedback, are moved to a release state in accord.

I don't understand your motives and you still haven't explained them after the last time we did a review.

I still have this one question: why is there a need for a second dark theme? And to add to the question would it not be better to work together?

So I need you to understand that this is not a personal attack on your work like you implied last time. It is a valid question for the sake of avoiding community split over (allegedly) equal plugins, duplication of efforts, avoiding confusion about which theme receives which bug report and most importantly why you see your work as important, what makes it unique and how willing you are to advance this in the next weeks, months and possibly years to come.

Only when you answer we can move forward with this, whichever direction fits best. But if you start acting up again, I will not start picking this up a third time. I hope we are clear.

Cheers,
Franco

@ghost
Copy link
Author

ghost commented Mar 27, 2018

First...Sorry, for my Bad english

So, its Version 1 because its ready and has no problems at all. I did a few version before but now it’s finished.
I did a lot of fixes, but the Problem was it never was accepted for the opnsense repo. Why? Only of the dark Team?
I did a lot of request on github but noone accepted my PR.

If you compare both themes, you will See that there are big differences in all.
Cicada is not Black and has thoughtful links and hovers. Its orange grey and I really struggled to get everything aligned. Links, hover, headings, search, health, so that a uniform picture is created and the people are happy. By the way. Did you Test my Theme yourself?
I tested also the dark team and in my personal opinion it has no concept.

The challenge for me was to make a Perfect Theme.it was a coincidence that this Theme and the dark team appear at the same time.
To make it short. Let the community decide which theme to choose and this should be adopted.

I can only say, that I like to make templates and I enjoy it. I would also like to continue the next years and make another complete different template in the future. But if you and the community don’t want it, then I keep this template for myself and delete all entries in the Forum.

I would like you to make a desicion now before investing more days in nothing.

Thx

@ghost
Copy link
Author

ghost commented Mar 27, 2018

So, what is your decision now?

@fichtner
Copy link
Member

You are impatient, impulsive and overly fixated on your personal work with little regard for cooperation and friendliness along the way. I'll save us both the time and ask you to drop this. I'm sorry.

@fichtner fichtner closed this Mar 27, 2018
@ghost
Copy link
Author

ghost commented Mar 27, 2018 via email

@fichtner
Copy link
Member

Sure, thanks.

@ghost
Copy link
Author

ghost commented Mar 27, 2018 via email

@fichtner
Copy link
Member

Not sure if you meant to send this. Remember: friendliness is key. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants