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

Undo New Log-Symbol in Current Nightly #15709

Open
wolverine007 opened this issue May 8, 2024 · 8 comments
Open

Undo New Log-Symbol in Current Nightly #15709

wolverine007 opened this issue May 8, 2024 · 8 comments
Labels
Feature Request A request for a new feature/function Unverified Issue not yet confirmed/reproduced or feature requests not yet checked for plausibility

Comments

@wolverine007
Copy link

Tell us your idea!

The former pencil has been replaced with a smiley in one of the last nightly releases.

Why?

Not all logs you will provide for a cache will result in a Found It = Smiley.

Placing a note or something else is not being reflected by a smiley, while an universal pencil is relfecting any log type.

What solution would you suggest?

Switch smiley back to former pencil

Is there an alternative?

no, the smiley is just irritating

Additional Information

No response

@wolverine007 wolverine007 added Feature Request A request for a new feature/function Unverified Issue not yet confirmed/reproduced or feature requests not yet checked for plausibility labels May 8, 2024
@bekuno
Copy link
Member

bekuno commented May 8, 2024

See #15619 and #15646

@rouroni1
Copy link

rouroni1 commented May 8, 2024

My two Cents about this subject.

And why not:
Revert to the pencil to do a log (found, dnf, etc...), and the option to select a list, use one icon with list with pencil?
Example:
https://pictogrammers.com/library/mdi/icon/playlist-edit/

After all when we press the bottom pencil icon, it's to change the list where we want to assign the geocache.

@fm-sys
Copy link
Member

fm-sys commented May 8, 2024

I agree that using the same icon for two actions is quite confusing. However, the "change lists" icon is the one which is confusing IMHO.

grafik

While the perfect material icon doesn't exist out of the box, I would recommend a mixture of those two material icons for "edit lists":

[images removed, see next post]


As a second step, we can discuss which icon to use for logging. May it be the old icon, a totally different one, or stay with the current one.

@fm-sys
Copy link
Member

fm-sys commented May 8, 2024

Edit: The material icon I was thinking of does actually exist, had just used the wrong keyword:

edit_note_24dp_FILL0_wght400_GRAD0_opsz24

@wolverine007
Copy link
Author

wolverine007 commented May 8, 2024

Edit: The material icon I was thinking of does actually exist, had just used the wrong keyword:

edit_note_24dp_FILL0_wght400_GRAD0_opsz24

I like the new variant for list management.

So we could stay with the well known pencil at both locations.

@wolverine007
Copy link
Author

The pen icon has always been used for different buttons (log and "edit" for several things (offline lists, note, ..)). On most devices there's no room for 4 toolbar icons which have been visible in CacheDetails for long time, so the use of the icon for logging is not that visible.

As for the icon, I'm using a smiley instead of the pen in my builds:
image

Maybe a different topic, but somehow linked with each other...

Any clue what is the magic number to get 4 icons in the pop up?

My device is offering a screen resolution of 2400x1080px, while a friend of mine is able to see 3120x1440px on his device.

He could see 4 icons (top pic), while my device is only able to show 3 icons.

But if you compare the two screens, you will see that there is plenty of room for a forth icon on my device as well:
PhotoCollage_20240508_132835203.jpg

@Lineflyer
Copy link
Member

Any clue what is the magic number to get 4 icons in the pop up?

AFAIK the Android system decides based upon criteria unknown to me (seemingly not only the resolution).
From your screenshots I guess also the text size is a bit different.

@fm-sys
Copy link
Member

fm-sys commented May 8, 2024

BTW, the TB activity still uses the old icon, which is pretty inconsistent. However, a Smiley wouldn't make sense for TB's...

grafik

fm-sys added a commit to fm-sys/cgeo that referenced this issue May 8, 2024
fm-sys added a commit to fm-sys/cgeo that referenced this issue May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request A request for a new feature/function Unverified Issue not yet confirmed/reproduced or feature requests not yet checked for plausibility
Projects
None yet
Development

No branches or pull requests

5 participants