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

Add storage info and no retry policy to share uploader #55

Merged
merged 9 commits into from
Nov 2, 2021

Conversation

HeDo88TH
Copy link
Collaborator

Added storage info popup

@HeDo88TH HeDo88TH self-assigned this Oct 26, 2021
@HeDo88TH HeDo88TH added the enhancement New feature or request label Oct 26, 2021
@HeDo88TH
Copy link
Collaborator Author

Ref DroneDB/DDB.js#3

@HeDo88TH
Copy link
Collaborator Author

Closes #52
Closes #47

@HeDo88TH HeDo88TH changed the title Add storage info Add storage info and no retry policy to share uploader Oct 26, 2021
package.json Outdated
@@ -19,7 +19,10 @@
"dependencies": {
"commonui": "file:vendor/commonui",
"copy-webpack-plugin": "^6.4.0",
"portal-vue": "^2.1.7",
"semantic-ui-vue": "^0.11.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we not add this? This package is not maintained and adds a lot of space just for displaying a popup.

Copy link
Collaborator Author

@HeDo88TH HeDo88TH Oct 28, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am using the popup component only. It works well and thanks to tree shaking It does not even add any meaningful weight to the web app:

webpack --mode=production

without library:
main.js	1.4MB	(2.9MB)

with library:
main.js	1.4MB	(3.0MB)

The first size is the compressed size transferred in the browser and the last is the uncompressed size.

I take full responsibility for this component: if it breaks I will re-write it from scratch 👍

It also packs a lot of useful Semantic components for Vue that may be useful in the future!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👎

You already have a modal setting from the Window component.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed in 00f82f4

package.json Outdated Show resolved Hide resolved
@pierotofy
Copy link
Contributor

Dependencies are still in package.json.

@HeDo88TH
Copy link
Collaborator Author

HeDo88TH commented Nov 1, 2021

Responsive behaviour is fixed in 5f40d9c

@pierotofy pierotofy merged commit 107b2fb into DroneDB:master Nov 2, 2021
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

Successfully merging this pull request may close these issues.

2 participants