Code Monkey home page Code Monkey logo

Comments (7)

philmtd avatar philmtd commented on June 30, 2024 1

I see. Yes I think Firefox does not have this built in... I will have a look at that library and think about how we could include this feature into the app. :)

from full-house.

philmtd avatar philmtd commented on June 30, 2024 1

Great. I just released it in version 1.3.0 (in case you use the Helm chart it's the chart version 1.2.1). Enjoy :)

from full-house.

Jisagi avatar Jisagi commented on June 30, 2024 1

Wonderful, thank you very much! I'm already running the latest version (using the docker version btw).

from full-house.

philmtd avatar philmtd commented on June 30, 2024

Hi and thank you for your suggestion!

Just to make sure I understand your use case: I guess that you share the game on a beamer and everyone participating in the meeting joins with their phones. Did I get that right?

I'm not 100% convinced that this feature will be generally useful to most users. Until today the use cases I know are people who use the game from their computers without sharing a screen or beamer so the link needs to be shared in a chat or other communication tool. I don't know which browser you use but some browsers (Chrome, Brave, ?) have a built-in feature to generate a QR code to the currently open website. Would that already work for you?

In case you really need the feature built in:

I like the QR code in the dialog better because it aligns better with the UI. Having it in the dialog would also allow you to always re-show the QR code to someone wanting to join later when there are already more than two players in the game.

In case we need to implement this we'll also have to make sure it integrates well into the design and maybe only shows the QR code if the user explicitly requests it to keep the UI clean. I'll be happy to help in that case.

But first could you please check whether you use a browser that already supports generating QR codes and if that would work for you? Let me know :)

from full-house.

Jisagi avatar Jisagi commented on June 30, 2024

Hi, thanks for your long answer.

Use case: Yes, that is exactly how we used to use the old planning poker we had. The projekt lead had the website projected to the wall and everyone either joined with a number&letter code or on mobile with the QR-Code.

I have seen the the qr generation in chrome, but switched to firefox just recently, not sure if I have seen the feature there. I didn't look into that too deep though, so I might have missed it. It's mostly a convenience thing and for those, who aren't as tech savvy as others. I don't particularly NEED this feature, but I thought, it might be a neat addition, so I suggested it.

The argument for the dialog option ist absolutely valid, I didn't even think about it no reappearing despite me even noting it in the initial post. Therefore scratch the other option.

The implementation I went for was done with angularx-qrcode, which is simple to use. Adding to what you said, the feature could be made optional globally maybe through the provided user config and even disabled in the default one. Different to that, a "show me a qrcode" button inside the invitation dialog could be an option as well, or a per-game toggle on creation.

When it comes to layout, I not a good frontend guy ;D I'd leave that to you. The code change for the generation is minimal and also uses "window.location.toString()" as input. The rest is just a element from the lib and a new line in the translation, that is it. I did not check how you implemented the global config, but I think that part is on the backend side of things, so a qr coding change would necessitate some kind of api change as well. The easiest option would therefore be something outside the config, maybe with the button or something, but you might have other ideas as well.

from full-house.

philmtd avatar philmtd commented on June 30, 2024

I created a first idea in #23. Would you mind to try it out and tell me whether it would fit your needs, @Jisagi?

from full-house.

Jisagi avatar Jisagi commented on June 30, 2024

This looks very nice and also does not intrude too much into the overall design. I never tried the config option for the lib, but did not think, such a clean look would be possible. I love it! If you like it, too, feel free to add this into the app.

image

from full-house.

Related Issues (2)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.