renyuneyun / Easer

User-defined explicit automation for Android
https://renyuneyun.github.io/Easer/
GNU General Public License v3.0
807 stars 93 forks source link

UI improvement #74

Open Ennoia11 opened 6 years ago

Ennoia11 commented 6 years ago

Hi,

How about dropping the top right menu ? In the Profile, Event and Scenario screen, you already have a "+" button that you can use. You might want to set it in the top of the screen and unify the color. Or perhaps replacing the round "+" button with a oval/oblong button with the text "Add" ?

For the Outline screen, you can make the big pink/green rectangle clickable and you already can access the about page through the left menu. screenshot_20180326-120917

renyuneyun commented 6 years ago

Hmm, thanks for the suggestions.

Yes, dropping the right-top menu makes sense. I forgot to remove as a part of the UI :P (Similarly, the "about")

For moving the "add": Don't you think moving the "add" button to the main frame takes too much space? Phone screens are already small, and why would we replace a non-space-taking element with a space-taking element with the same function (and same semantics)?

For changing color: I'm not very good at picking colors, but one intuition is that the current color of the button doesn't destroy the whole while also highlights the button. (But the central color of the button may need to be changed.)

For the outline page: do you suggest clicking the indicator (big rectangle) to toggle on/off of the service? Making everything clickable may be an interesting UI design, but it's not obvious... I'm planning to use cards to replace these, and it may be possible to add some actions to cards (when "expanded").

Ennoia11 commented 6 years ago

I wouldn't agree that smartphone screen are too small but that it a personal opinion.

OK, if you're want space saving, why don't replace the top right menu with the "+" button ? A menu makes sense with several elements which is not the case. Is it technically possible ? Or are you planning to add something else in the menu ?

For the color, I didn't see this bottom button in the first place even with its flashy color ! I went right for the menu and I'm not color-blind. I'm reading top to bottom, right to left, so that probably why this is the last place I'd check.

And for the outline page, yes, toggling the service by hitting the indicator is exactly what I meant. I'm not sure what you mean by cards but I'll probably understand when you'll release it !!

natrius commented 6 years ago

I like the "+"-button in the right bottom corner a lot. To add - i would also make the Button in the Outline clickable. Was the first thing i tried tbh :D Also, write the "Running" or "Not running" directly in the button, so it consumes less space. I also don't know what is meant with "cards".

renyuneyun commented 6 years ago

@natrius That button in the "Outline" page is indeed clickable, but it's function is a bit strange now (but wan't that strange initially). It will force the background service to reload (from disk) and re-check everything (which doesn't make sense for what Event means now).

natrius commented 6 years ago

In this Case, make the button small and rename it to "Refresh status" or something. And, put a new line on top of "Profile, From event, Loaded time", called "Status: Running" with a green colour and "Not running" with a red colour.