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

NOTICE: Floorp Browser Progressive Web Apps #819

Closed
surapunoyousei opened this issue Nov 14, 2023 · 24 comments
Closed

NOTICE: Floorp Browser Progressive Web Apps #819

surapunoyousei opened this issue Nov 14, 2023 · 24 comments
Labels
Browser::Fork Changes or bugs from Fork browsers Class::Icon Icon for panels, menus, libraries Issue::Enhancement New feature or request

Comments

@surapunoyousei
Copy link
Contributor

Is your feature request related to a problem?

Just a heads up. As you may know, I am currently adding a PWA implementation to Floorp. This requires some icons.

I am currently replacing the pre-Firefox Proton icons, but the design may not match and the Lepton look may be lost.

We have contacted you to see if a new icon is needed.

The solution I'd like

Consider create icons

Screenshots or video

image
image

Alternatives I've considered

No response

Additional context

No response

@surapunoyousei surapunoyousei added the Issue::Enhancement New feature or request label Nov 14, 2023
@black7375
Copy link
Owner

IMO, You can refer fluent icon.

https://fluenticons.co/outlined

image
image


I've been working a full-time job and going to univercity lately, so I don't have time during the week.
I'll check it out a bit more over the weekend.

@surapunoyousei
Copy link
Contributor Author

I am a student like you, so I understand that you are busy.

Take your time and everything will be fine. I do not intend to release this feature any time soon, the main thing is to keep you informed.

@surapunoyousei surapunoyousei changed the title NOTICE: NOTICE: Floorp Browser Progressive Web Apps Nov 14, 2023
@zapSNH
Copy link
Contributor

zapSNH commented Nov 17, 2023

I might be able to make some icons for it.

--
edit:
see:
https://www.figma.com/file/Iyym2tjRVlb7rxTR1Gmvr8/lepton?type=design&node-id=154%3A2&mode=design&t=a8V3q0FF83RF9L48-1

There are two variations of the icons, window and square. Let me know which one you like and if you have any feedback on these icons.

@black7375
Copy link
Owner

IMO, Although the last one seems hard to recognize as an PWA Apps,
I think these are fine for installing and deleting PWAs.


@surapunoyousei What do you think about @zapSNH icon?

Maybe it'll help PWAsForFirefox too.
cc @filips123

@surapunoyousei
Copy link
Contributor Author

surapunoyousei commented Nov 19, 2023

I might be able to make some icons for it.

-- edit: see: https://www.figma.com/file/Iyym2tjRVlb7rxTR1Gmvr8/lepton?type=design&node-id=154%3A2&mode=design&t=a8V3q0FF83RF9L48-1

There are two variations of the icons, window and square. Let me know which one you like and if you have any feedback on these icons.

IMO, These icons, users may misread to be "window" icons. All currently icons are difficult to recognize.

image

Good one is Microsoft Edge's icon (Fluent). Maybe it needs 4 squares in icons

image

@zapSNH
Copy link
Contributor

zapSNH commented Nov 19, 2023

IMO, These icons seem to be "window" icons. All currently icons are difficult to recognize.

There's also a square version on the right.

Good one is Microsoft Edge's icon (Fluent). Maybe it needs 4 squares in icons

image

The 4 square icon kinda looks too similar to the container icon and it may confuse people. I can whip up a 4 square version though if you want. (right after i eat my lunch)

image

@surapunoyousei
Copy link
Contributor Author

IMO, These icons seem to be "window" icons. All currently icons are difficult to recognize.

There's also a square version on the right.

Good one is Microsoft Edge's icon (Fluent). Maybe it needs 4 squares in icons
image

The 4 square icon kinda looks too similar to the container icon and it may confuse people. I can whip up a 4 square version though if you want. (right after i eat my lunch)

image

OK. I agree with you. I forget about that.

Looks like we'll have no choice but to compromise...

@black7375
Copy link
Owner

black7375 commented Nov 19, 2023

It certainly looks so similar.


Fortunately, I have a couple of alternatives.

Icons

Cube

image

Grid

You have the option to make it a dot or a rectangle.
image image

Variants

I estimate that we'll need four icons.

  • PWA Apps (Panel item or toolbar button)
  • PWA Apps install (URL bar before installed)
  • PWA Apps launch (URL bar after installed)
  • PWA Apps delete (At popup)

Deform the model itself

image

Mode in the lower right corner

image

Combination of one icon and another

In Chrome, only the Apps icon is grid, and install utilize the existing icon.

image

image

You might also want to consider the Install button in PWAsForFirefox.
image

@zapSNH
Copy link
Contributor

zapSNH commented Nov 19, 2023

I've made something similar to Chrome's image (and the PWAsForFirefox's) icon.

image
launch

What do you think about it?

@surapunoyousei
Copy link
Contributor Author

surapunoyousei commented Nov 19, 2023

In conclusion, the first icon you presented seemed to be a good choice.

#819 (comment)

@zapSNH
Copy link
Contributor

zapSNH commented Nov 19, 2023

In conclusion, the first icon you presented seemed to be a good choice.

What version? Window or square?
image

@surapunoyousei
Copy link
Contributor Author

In conclusion, the first icon you presented seemed to be a good choice.

Square are better

@zapSNH
Copy link
Contributor

zapSNH commented Nov 19, 2023

I've added the square icons zapSNH/lepton-icons but I can change it if @black7375 wants a different icon.

@black7375
Copy link
Owner

black7375 commented Nov 19, 2023

Wow, Launch icon is beautiful!!
image


Currently, the app icon looks like a single app, so it's better to make it look like you have multiple apps.
My idea is to place the dots inside a regular desktop icon with a border.

image

It's probably consistent with mobile as well.

@zapSNH
Copy link
Contributor

zapSNH commented Nov 19, 2023

How about this for the app icon (I assume you mean the pwa-manage icon)
image

@black7375
Copy link
Owner

Nice 👍 I like it.

@zapSNH
Copy link
Contributor

zapSNH commented Nov 19, 2023

I've pushed the svgo'd icons to zapSNH/lepton-icons so you can get them there.

@black7375
Copy link
Owner

As soon as the CSS selector names in Floorp are stable, I'll add the icons. @surapunoyousei

@surapunoyousei
Copy link
Contributor Author

Ok. I'll send infomation here.

@surapunoyousei
Copy link
Contributor Author

surapunoyousei commented Nov 29, 2023

The most of implementation of PWA has been completed and additional features may appear in the future, but we will report them once they are in place.

Please follow below CSS id & attribute.

"open-ssb" means, Not install page. The button will open installed PWA window(ssb window)

/* Panel UI top button */
#appMenu-ssb-button {
  list-style-image: url("chrome://browser/skin/install-ssb.svg");
}

/* Panel UI subView button  for install ssb*/
#appMenu-install-or-open-ssb-current-page-button {
  list-style-image: url("chrome://browser/skin/install-ssb.svg");
}

/* Panel UI subView button  for open ssb window*/
#appMenu-install-or-open-ssb-current-page-button[open-ssb="true"] {
  list-style-image: url("chrome://browser/skin/install-ssb.svg");
}

/* Page Action button. Not always visible */
#ssbPageAction-image {
  list-style-image: url("chrome://browser/skin/install-ssb.svg");
}

/* Page Action button. If page has been already installed or can be install PWA */
#ssbPageAction-image[open-ssb="true"] {
  list-style-image: url("chrome://browser/skin/install-ssb.svg");
}

black7375 added a commit that referenced this issue Dec 2, 2023
@black7375
Copy link
Owner

I made a commit, check it out

@surapunoyousei
Copy link
Contributor Author

surapunoyousei commented Dec 6, 2023

Add a separate icon by @zapSNH .

I thought these would be included in Lepton but, I guess These not include it in Lepton.

I will create pull request

@black7375 black7375 added Class::Icon Icon for panels, menus, libraries Browser::Fork Changes or bugs from Fork browsers labels Dec 7, 2023
@black7375
Copy link
Owner

I think we can close this issue now!!

@surapunoyousei
Copy link
Contributor Author

Oh... I forgot it... My apologize.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Browser::Fork Changes or bugs from Fork browsers Class::Icon Icon for panels, menus, libraries Issue::Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants