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

feature requests #131

Closed
jsarenik opened this issue Dec 17, 2019 · 9 comments
Closed

feature requests #131

jsarenik opened this issue Dec 17, 2019 · 9 comments
Labels
enhancement New feature or request

Comments

@jsarenik
Copy link

Putting down some of my thoughts and waiting for feed-back to see if my thoughts are better fit for a custom fork or would be accepted generally.

  1. No external assets (currently there are some css files being loaded from
  2. Send message on ENTER (or at least CTRL+ENTER)
@nwtgck
Copy link
Owner

nwtgck commented Dec 18, 2019

Thanks again @jsarenik! I think ctrl + enter is a good idea too. In macOS, it should be cmd + enter.

I'm not sure the meaning of "1 No external assets". Could you explain it more?

@nwtgck nwtgck added the enhancement New feature or request label Dec 18, 2019
@jsarenik
Copy link
Author

Sure. I will explain.

  1. Open a new tab in Chromiuml/Chrome (note that piping-chat-web does not work in Firefox)
  2. Open Developer Tools, Network tab.
  3. In browser tab open https://xn--jrok-loa.bublina.eu.org/chat/ (which is a build of piping-chat-web with some updates and a changed serverUrl of the piping-server. See https://github.com/jsarenik/piping-chat-web/tree/jsn/bublina)
  4. Look into the list of 6 assets loaded. See mine below.
https://xn--jrok-loa.bublina.eu.org/chat/
https://d33wubrfki0l68.cloudfront.net/bundles/df13aee16387d94666662df35e6f9d3b4d7980f2.js
https://d33wubrfki0l68.cloudfront.net/308ee3580a4a34bd230025d460762d53b5344efc/b5f29/chat/fonts/materialicons-regular.0509ab09.woff2
https://xn--jrok-loa.bublina.eu.org/chat/manifest.json
https://xn--jrok-loa.bublina.eu.org/chat/img/icons/favicon-32x32.png
https://xn--jrok-loa.bublina.eu.org/chat/img/icons/android-chrome-192x192.png

The only external resources are second and third.
The second line points to vuejs and the third line points to materialicons font.
Both can be made local so that they will not be loaded from cloudfront.net.
Then piping-chat-web can be used even in evironments with no access to cloudfront.net.

@jsarenik
Copy link
Author

@jsarenik
Copy link
Author

Thanks again @jsarenik! I think ctrl + enter is a good idea too. In macOS, it should be ctrl + cmd.

CMD+ENTER

@nwtgck
Copy link
Owner

nwtgck commented Dec 18, 2019

@jsarenik
Thanks. "1 No external assets" means you propose piping-chat-web should not load external resources. Is it right?

@jsarenik
Copy link
Author

Yes. It would be definitely great as an option at least (I mean without manual editing which is possible even now if I was able able to see that URL with cloudfront.net somewhere inside the generated files. My guess is that js/chunk-vendors.17861028.js is doing the magic...).

Thank you @nwtgck

@nwtgck
Copy link
Owner

nwtgck commented Dec 18, 2019

@jsarenik
I think I can remove external assets. Actually, another project using Piping Server, Piping UI is independent of external assets. Piping UI never accesses to external assets such as icons and Workbox for higher privacy/security purposes, but icons and PWA (Workbox) are available. So, I may implement the feature to use the knowledge from the development probably.

You can create two new issues for 1 and 2. Then, I will close this issue. Of course, I will create issues instead of you when you don't create.

@jsarenik
Copy link
Author

You can create two new issues for 1 and 2. Then, I will close this issue. Of course, I will create issues instead of you when you don't create.

Sure, I will do it. Thank you!

This was referenced Dec 18, 2019
@jsarenik
Copy link
Author

Closing in favor of #133 and #134

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants