feat(interactables): user picker component #3506
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
Missing Dev Review
✅ Yeeeehaw, deploy preview is ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
josephmcg
suggested changes
Jun 8, 2022
josephmcg
added
Changes Requested
josephmcg
approved these changes
Jun 8, 2022
josephmcg
added
Ready for QA
|
making a note that we want to add some sort of list to keep track of currently selected users. thought it would be best in a separate PR to keep things simple |
|
Found when testing, seems the group avatars will overlap with the new group modal
|
phillsatellite
added
Changes Requested
|
@phillsatellite fixed :) |
stavares843
added
Ready for QA
|
thanks @jasonwoodland 🔨 |
stavares843
force-pushed
the
AP1742
branch
2 times, most recently
from
1170431
to
869c411
Compare
phillsatellite
approved these changes
Jun 9, 2022
phillsatellite
added
QA tested
|
very neat 🎉 i noticed that it doesn't show feedback if the search doesn't have results and the user tries to click on chat now even if the button is already not tinted but maybe we could improve that on other PR
this branch vs dev where we don't show the button if there are no results but this component is way prettier 🎉 cc @josephmcg |
stavares843
approved these changes
Jun 9, 2022
github-actions
bot
removed
the
QA tested
JustZacca
pushed a commit
that referenced
this pull request
Jun 10, 2022
stavares843
reviewed
Jun 10, 2022
| @@ -0,0 +1,16 @@ | |||
| <div class="user-picker"> | |||
There was a problem hiding this comment.
this is adding a warning when we build locally, could fix you on a separate PR? @jasonwoodland 🔨
There was a problem hiding this comment.
added ticket here - AP-1766
aewing
added a commit
that referenced
this pull request
Jul 7, 2022
* chore(cypress): skip tests * feat(chat): implement infinite scroll for chat messages (#3286) * chore(lint): minor lint (#3542) * fix(toolbar): add missing user avatar src prop (#3545) * fix(layout): remove overflow hidden (#3529) * fix(call): add possibility to call user via context menu (#3484) * chore(copy): minor update * fix(call): overflow for long user name in incoming call (#3499) * feat(interactables): user picker component (#3506) * feat(jest): 100% coverage for store/accounts/mutations (#3554) * refactor(scrollbar): spread new scroll component (#3284) * chore(seedPage): added copy button to seed page (#3543) * fix(buckets): initialize buckets separately (#3547) * chore(comment): add comment + minor lint (#3571) * chore(jest): add jest tests (#3569) * chore(lint): lint (#3572) * chore(cypress): fix cypress test (#3573) * chore(cleanup): remove unused markup (#3528) * fix(video): fix toggle video bugs, improve video unit tests (#3490) * chore(jest): add more jest tests (#3574) * chore(locales): update copy * chore(jest): remove not used import on test (#3577) * chore(lint): minor lint (#3576) * feat(jest): fix lint errors (#3587) * feat(userpicker): add chip list (#3552) * chore(jest): update jest (#3596) * fix(call): show avatar placeholder on incoming call (#3556) * chore(cypress): skip test (#3606) * feat(chat): add message when a user adds somebody to chat (#3608) * fix(call): destroy call on decline incoming call (#3558) * feat(jest): 80% coverage for store/ui/mutations (#3616) * feat(jest): 100% coverage for store/ui/getters (#3615) * feat(jest): 100% coverage for store/chat/mutations (#3621) * feat(jest): 47% coverage for libraries/WebRTC/Emitter (#3623) * chore(edit): hide edit message (#3620) * fix(glyphs): prevent glyph drag (#3609) * fix(chat): correct avatar color for DM (#3586) * fix(call): add exit call sound (#3612) * chore(imports): remove not used imports (#3598) * chore(remove): remove not needed stuff (#3622) * fix(call): add sound on end call on caller side (#3614) * fix(double-requests): removed double friends requests (#3590) * refactor(scrollbar): clearing remaining components (#3514) * chore(loading): removed loading screen on chat (#3551) * fix(profile): simplify profile, minor refactor (#3610) * fix(chat): refresh crash error * fix(reply): removed reply to replies (#3626) * fix(friends): name and status overflow issue (#3640) * fix(userpicker): friend name overflow (#3637) * fix(focus-chat): fixed auto focus when entering chat (#3642) * fix(call): fullscreen call search results z index issue (#3641) * chore(cypress): skip test * chore(cypress): minor update * chore(cypress): comment out test * chore(cypress): skip test * chore(cypress): skip test * chore(cypress): comment out test * fix(chat): is active call logic (#3636) * feat(toolbar): add coming soon to group search input (#3658) * fix(profile): add flair color to profile picture upload, change size (#3660) * fix(select): broken select rendering (#3657) * fix(quickchat): improve quickchat/sidebar css (#3588) * fix(toolbar): remove secondary create group modal (#3585) * chore(jest): fix lint errors on SearchQuery (#3673) * feat(jest): 95% coverage for store/friends/getters (#3672) * fix(groupCreateForm): allow enter on name input to create group (#3581) * fix(account): client side validation for user creation (#3639) * fix(cta): svg overflow clipping (#3670) * feat(message): disable emoji/reply in group chat (#3655) * chore(cypress): add commands locally to run tests (#3684) * fix(mobile): bottom nav bar state (#3662) * chore(lint): minor lint (#3685) * chore(profile): hide profile (#3687) * fix(call): add sounds for call (#3644) * chore(update): update locale * fix(onlineStatus): update peer events handlers (#3557) * chore(cypress): comment out test * chore(cypress): skip test * fix(media): change max-width of user name (#3728) * fix(quick): break word (#3719) * feat(jest): 46% coverage for store/chat/actions (#3727) * chore(jest): remove unnecessary arguments (#3725) * chore(cypress): update * fix(sidebar): proper color on sidebar tag" (#3710) * fix(chat): unknown name while loading (#3714) * fix(groups): fix context menu for groups sidebar item (#3723) * refactor(toolbar): refactor styles and increase text overflow length w/ flex (#3721) * fix(files): remove unfinished ux (#3709) * fix(chat): link styling (#3713) * fix(font-family): aligned apps fonts (#3671) * fix(mobile): settings menu toggle position (#3720) * chore(groups): add back click action (#3730) * chore(cypress): add friend test (#3735) * fix(chat): is_typing indicator (#3668) * fix(double-requests): filtered incoming requests (#3686) * fix(state): add on discover event (#3649) * fix(groupChats): add retry to invite search in group creation (#3667) * chore(update): update read me * chore(cypress): minor update * chore(cypress): skip test * fix(userpicker): style & overflow issues (#3726) * chore(lint): minor lint (#3747) * chore(cypress): skip test * Update package.json bump version * fix(download): remove unnecessary local state, fix string bug (#3707) * feat(groupinvite): use userpicker component (#3607) * feat(jest): 97% coverage for store/textile/mutations (#3765) * feat(jest): 100% coverage for store/webrtc/mutations (#3762) * fix(files): make filesystem progress/total size reactive (#3754) * fix(mobile): mobile nav style (#3755) * chore(jest): add test (#3768) * fix(satellitecircle): remove illegible light grey color (#3769) * fix(chat): upload rework (#3550) * fix(media): only flip local video stream (#3770) * feat(select): new keyboard accessible select component (#3669) * feat(jest): 100% branch coverage for store/webrtc/mutations (#3774) * feat(jest): 19% coverage for SharedBucket (#3776) * fix(mobile): settings swiper flow (#3715) * test(cypress): add tests for videocall pair chat (#3331) * fix(sidebar): subtitle text overflow (#3758) * fix(conversation): offline status (#3767) * fix(groupchat): fix for username of unknown users during initial load (#3766) * test(cypress): updates to chat top toolbar tests (#3781) * fix(sendmoney): disabled button prop (#3761) * feat(jest): 97% coverage for store/textile/mutations (#3791) * feat(notifications): add browser alert and notification image (#3538) * fix(friends): online status is hidden if not friend (#3680) * chore(lint): minor lint (#3793) * chore(typo): fix typo * test(cypress): improvements to avoid failure on last test (#3797) * fix(search): only pull results from current chat (#3789) * fix(webrtc): on call track logic (#3790) * fix(reply): hide emojies on threads (#3763) * chore(store): remove unused store (#3787) * fix(select): cleanup select console errors (#3786) * fix(chatbar): change chanbar z-index depend on fullscreen media (#3764) * fix(vscode): default imports to be from project root (#3773) * test(cypress): fixes for chat images validations (#3795) * chore(cypress): remove skip * chore(cypress): remove skip * chore(cypress): remove skip * chore(cypress): remove skip * fix(group): remove duplicated dispatch to storeInMessage action (#3796) * test(store): create mock root state that will automatically keep in sync (#3759) * chore(changelog): update changelog (#3805) * feat(logs): add logs to help monitoring webrtc activities (#3779) * fix(textile): set peer id correctly when switching conversations (#3803) * fix(chat): reaction buttons style and ui (#3757) * chore(cypress): minor update * test(cypress): updates on mobiles and pair tests (#3813) * fix(quickchat): z index fix for quickchat modal (#3814) * feat(jest): 92% coverage for utilities/Messaging (#3815) * chore(jest): remove useless console log (#3832) * feat(jest): 84% coverage for Item.abstract.ts (#3833) * feat(jest): 100% coverage for store/conversation/getters (#3831) * feat(jest): 96% coverage for Item.abstract.ts (#3844) * fix(settings-modal): fixed modal style (#3560) * test(cypress): fixes for text and videocall tests (#3838) * test(cypress): updates to privacy settings tests (#3851) * chore(cypress): add downloads folder * chore(cypress): skip test * chore(cypress): skip test * chore(cypress): skip test * feat(jest): 20% coverage for Thumbnail (#3864) * feat(jest): 19% coverage for BlockchainClient.ts (#3863) * feat(newmessage): use userpicker (#3643) * fix(chat): break word on chat replies (#3841) * chore(cypress): delete * chore(message): remove unused timestamp variables (#3830) * chore(lint): minor lint (#3866) * chore(cypress): update cypress to 10.3 version (#3868) * refactor(slimbar): improve styles and markup (#3826) * fix(friends): prevent 'add friend' picture shrinking (#3842) * feat(friends): can click on a friend and open direct message * fix(search): add markdown to search results (#3840) * fix(notifications): text overflow for notifications (#3839) * fix(user-name-style): fixed name lineheight (#3884) * feat(jest): 100% coverage for store/chat/mutations (#3904) * style(chat): fix scrollbar overlapping with long messages in chat (#3883) * fix(chatbar): control buttons become grey when user cant interact * refactor(mediauser): styling and fix indicator color (#3878) * feat(dragbar): add dragbar implementation (#3860) * feat(friends): friends WIP * feat(friends): friends WIP, env vars * fix(files): keyboard accessible, hit esc to close (#3847) * feat(call): create background call component (#3591) * feat(friends): friends WIP, cleanups * fix(chatbar-mobile): fixed chatbar focus when pressed (#3777) * feat(friends): chat WIP, progress * chore: disable old tests that are failing * feat(friends): chat WIP, ChatManager * test(cypress): fix or skip failing tests (#3892) * chore(githubAction): add iridium label automatically (#3910) * chore(fix): fix github action * chore(update): minor update * chore(update): remove github action * chore: WIP integration * chore: WIP integration - friend request * chore: WIP integration - friend request registration bug * chore: disable old tests that are failing * chore(package.json): add iridium via yarn * chore: fix the build process * fix: update iridium dependency to fix the build * fix(files): fix files embed overflow issue (#3924) * fix(sidebar): add relative timestamps to sidebar (#3926) * feat(chat): friends and chat progress * feat(chat): friends refactor wip * feat(chat): friends requests and chat progress * fix(audio): error handling for missing audio output devices (#3925) * fix(audio): error handling for missing audio output devices * refactor(audio): rename audio variables * refactor(audio): remove console log * feat(settings): add page and section components (#3928) * feat(settings): add page and section components * feat(settings): update settingsunit styling * fix(interactables): volume margin * feat(settings): replace bulma with settings components * fix(settings): remove unused style * fix(settings): typo * fix(settings): remove br tag * fix(select): max-width overflow * chore(webrtc): actions updated with iridium * feat(files): unblock files ui, basic upload and download (#3929) Co-authored-by: Sara Tavares <29093946+stavares843@users.noreply.github.com> Co-authored-by: Sasha Zakablukov <38879804+AlexZakablukov@users.noreply.github.com> Co-authored-by: Jason Woodland <me@jasonwoodland.com> Co-authored-by: Andre Christoga Pramaditya <caxvis@gmail.com> Co-authored-by: Thomas Bosatelli <58808132+ThomBos@users.noreply.github.com> Co-authored-by: Chris Hogan <chris@hogan.io> Co-authored-by: Joe McGrath <33670767+josephmcg@users.noreply.github.com> Co-authored-by: Muhammad Al Juburi <maljuburi89@gmail.com> Co-authored-by: pavlzk <97599870+pavlzk@users.noreply.github.com> Co-authored-by: molimauro <50131258+molimauro@users.noreply.github.com> Co-authored-by: Luis Cardeña <35935591+luisecm@users.noreply.github.com> Co-authored-by: Andrew Ewing <drew@phenocode.com> Co-authored-by: Jeff Kristian <jekrimo@gmail.com> Co-authored-by: vimercati-samir <52865716+vimercati-samir@users.noreply.github.com> Co-authored-by: Mauro Molinari <mauro.molinari@knobs.it> Co-authored-by: Drew Ewing <drew.ewing@satellite.im> Co-authored-by: Nathan Power <Nathan.c.power@icloud.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does 📖
typeprop in quick componentWhich issue(s) this PR fixes 🔨
AP-1742
AP-1710
Special notes for reviewers 🗒️
Additional comments 🎤