Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

icysapphire's Fix for people appearing appearing on top of each other #282

Open
wants to merge 31 commits into
base: master
Choose a base branch
from

Conversation

pisanvs
Copy link

@pisanvs pisanvs commented Jul 15, 2020

By submitting this Contribution you hereby grant to Chris Gamble a worldwide, royalty-free, non-exclusive, perpetual and irrevocable license, with the right to transfer an unlimited number of non-exclusive licenses or to grant sublicenses to third parties, under the Copyright covering the Contribution to use the Contribution by all means, including, but not limited to: publish the Contribution, modify the Contribution, prepare derivative works based upon or containing the Contribution and/or to combine the Contribution with other Materials, reproduce the Contribution in original or modified form, distribute, to make the Contribution available to the public, display and publicly perform the Contribution in original or modified form.

note: Everything here was made/fixed by icysapphire, not by me.

"author": "Chris Gamble",
"homepage_url": "https://github.com/Fugiman/google-meet-grid-view",
"version": "1.39.0",
"author": "Simone Marullo",
Copy link

Choose a reason for hiding this comment

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

The changes in the manifest file should probably be pulled out before merging.

Copy link

@ajs256 ajs256 left a comment

Choose a reason for hiding this comment

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

The funding info should probably be changed before this gets merged.

@ajs256
Copy link

ajs256 commented Sep 9, 2020

And also, the menu no longer works.
image
When I hover over the icon, the menu still appears at the usual place, making it unusable.

@buenoudg
Copy link

buenoudg commented Sep 9, 2020

And also, the menu no longer works.

Using @icysapphire's fix in Firefox, I also have the button in the wrong place, but the menu works as always.

buenoudg and others added 4 commits September 11, 2020 19:59
Two new strings were added in version 1.40: 'Current release' and 'Original release here (discontinued)'

This changes them to localized strings, with versions in English, Spanish, and Catalan
Some localizations (en, es, ca) for new strings
Some spanish localization fixes
@apalumbo
Copy link

apalumbo commented Dec 3, 2020

@icysapphire would you mind to change the manifest stuff so that the PR could be merged?

Copy link
Author

@pisanvs pisanvs left a comment

Choose a reason for hiding this comment

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

should add this...
edit: Didn't notice I'm reusing this PR

@Jkarhani
Copy link

The grid is no longer working well, all the people pictures are stuck above others..Any help please?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants