-
Notifications
You must be signed in to change notification settings - Fork 9
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
AppCenter Submission: App Not Accepted #4
Comments
Hey, thanks for the information. Now, I have two more questions, What if I decide to add more functionality to the application in parallel with the indicator? Like a process monitor that has also an indicator? |
@PlugaruT the primary reason is that the system indicator area by design provides first-party, OS-wide indicators. Opening the flood gates to allow third-party apps to add their own indicators means we're inviting a ton of clutter and making that space less purposeful. I cover this a bit in this developer tips blog post: https://blog.elementary.io/developer-tips-backgrounding-system-integration/
Currently I don't believe that would be accepted, because the problem comes down to adding an app indicator to the system area. |
Yeah, I get that and I partially agree, because it all comes down to what the users wants, if the user wants to add something, you bet he will find a way to do this. |
@cassidyjames What is the difference between my Monitor Indicator and @PlugaruT's indicator? Is this because I ship an indicator with an app? |
@stsdc your app probably probably predated our exact clarification of the rules so you may have been grandfathered in. Otherwise, my answer would be: we're human and we missed it. |
@cassidyjames, what are the chances you would reconsider the decision? |
Quoting a comment I made recently in StackExchange about this topic:
Isn't displaying system load precisely that sort of OS-wide indicators? I don't see why it would be necessary to automatically block all apps from using indicators; since there is already a review step for AppCenter apps, it seems to be sufficient to apply the criteria defined in the HIG during the review... |
@cassidyjames, I honestly never ran a linux distro without adding a system monitor indicator. I currently run Elementary OS on an Asus T101HA which is an Intel Atom with 4GB of RAM. So I definitely need such an extension to constantly see if I'm on the edge of my PC's resources. I'd actually like to see such an indicator as an OS feature which could just be enabled from System Settings. |
Of course if @PlugaruT agrees, @cassidyjames, I know that are probably very busy working on Odin but I wanted to ask if would you guys consider adopting this system monitor indicator into eOS as a first-party, OS-wide indicator? I think an indicator like this is really needed and provide it as part of the OS will be great so we users don't have to go hunting for the same in other places, which likely end up breaking eOS panel look and feel anyway. Just my two cents. PS: I would consider splitting the weather feature into a different, dedicated, weather indicator and leave this one dedicated to monitor different system resources only. |
@casasfernando I'm not sure; there hasn't been a huge demand for a resource monitor, and even if we included one, I don't think we'd include it as an indicator—it would probably just be an app. Regarding Weather, there is an open issue and PR against the Date & Time indicator: elementary/wingpanel-indicator-datetime#255 |
I would just like to add an unsolicited opinion and say that this restriction on the App Center is arbitrary and users are smart enough to know clutter from utility. Especially when they purposefully download and install an application like this one, whose sole function is to be an indicator of system resources. OS-level, you might say. This removal of Ayatana and other Wingpanel indicators is why I had to begrudgingly switch away from eOS a couple of years ago. I get that you guys have a vision for no icons up there, but like, don't install them on your machine if you dislike them so much. They're clearly wildly popular on macOS and other Linux Distros. |
@cassidyjames thanks a lot for taking the time to get back to my comment. I forked @PlugaruT 's project to https://github.com/casasfernando/wingpanel-indicator-weather as a dedicated weather indicator, did some modifications and add a few features (including manual location setting). Many thanks for @PlugaruT since his code was a great base to work on. |
Why not create a PPA on launchpad as a workaround? See #27 (comment) |
OK Odin is out, what's the follow on this? |
Hey @PlugaruT, thank you for another app submission to AppCenter! However, after a review of your app, we have determined that it is an “extension” or “plugin” of the Panel. As listed in the Publishing Requirements:
At some point in the future, we may expand the types of apps accepted into AppCenter, and we will announce any such major changes on our blog. We hope this does not deter you from submitting future apps that are supported!
Let me know if you have any questions and I would be happy to answer them.
The text was updated successfully, but these errors were encountered: