offline-qr-code icon indicating copy to clipboard operation
offline-qr-code copied to clipboard

Add a menu to the popup

Open jamescallumyoung opened this issue 6 years ago • 3 comments

Background

In #213 we have discussed how we think a menu should be displayed so that extra features (in this case, special forms) can be added to the popup without cluttering the UI.

I'm proposing this separate feature request so we can keep the discussion about the menu here and keep #213 focused on the feature requested.

Proposed solution

Add a small button along the top of the popup window that opens up a menu. The menu will probably be best if it covers the whole popup as the window is pretty small.

Links to extra pages within the popup, and maybe other options, can then be added to the menu.

Alternatives

I originally suggested a "material" themed header and sidebar solution but @rugk suggested this might take up too much space on the small popup window.

Additional context

See #213 for the earlier conversation.

jamescallumyoung avatar Oct 16 '19 11:10 jamescallumyoung

Indeed, good idea to open a new issue.

The menu will probably be best if it covers the whole popup as the window is pretty small.

I doubt so.

Did a quick mockup and I actually had a "show all options, hover over one and it shows the option" in my mind. This also solves the space problem a little, because you can only show an icon, when it it not hovered, and "expand" the option, if it is shown.

grafik

menuPopup.zip

(created with drawIo)

rugk avatar Oct 16 '19 12:10 rugk

I would like to handle this issue. I would add buttons for saving and copying the QR code like it was mentioned in the #267 issue.

jednomije avatar Oct 08 '22 10:10 jednomije

@jednomije You may give it a try, but please have a look at the previous discussion and the mockup/ideas. We need to come up with some clever UI that does not cover the QR code, but does not waste space, too.

(And sorry for the late reply, I was busy and missed the notification, sorry.)

rugk avatar Oct 19 '22 19:10 rugk