-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Room header & details UI refresh (+ what's needed to remove from Labs) #25883
Comments
Also copying from the previous issue: Is #25534 invalid as we're no longer providing a drop-down header? @nadonomy or @americanrefugee this is a question for you :) |
Correct @daniellekirkwood - there is no longer a drop-down menu from the room header. Click to expand the right sidebar only. |
Closed this as it's ambiguous. Tough to action. We can get fresh insights once the new implementation is testable @americanrefugee for this I think we should stress test in Figma and propose the smallest width we think works. @daniellekirkwood would you be able to pull some analytics insights on this pls? Knowing what resolutions are in use in the wild would help. We could also formalise this in the project README (with browser support etc) for posterity. Commenting there - @americanrefugee your input is welcome too. |
@germain-gg updated the issue tasklist to include https://github.com/vector-im/element-internal/issues/444 which details how we should solve notifications. |
@germain-gg on I checked analytics and there's little useful to discern on this one. So, I think let's remove it and monitor for any feedback we get on develop after we merge. |
I never liked defaults with no member list visible. Now when you removed feature_right_panel_default_open, i have no way to fix it. 👎👎👎 |
@germain-gg & @janogarcia / @americanrefugee I don't have enough familiarity with screen widths to be able to read this chart... however, as asked for here's some data! |
We need to accomodate this matrix-org/matrix-react-sdk#11475 in the new room header too Looping @charlynguyen in as the developer who's been looking at the knocking feature |
Worth tracking these two issues: Note: Compound was sold in order to improve accessibility throughout the product. We should ensure that we continue to drive that focus via these components. |
I've estimated a day for each of the bugs on the room details list here (not including adding the missing options) so let's say this is about Edit: actually, scratch that, there might be some compound work involved with at least some of the components, so make it two weeks. |
* Update all non-major dependencies (#26201) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Michael Telatynski <[email protected]> * Update build.yml * Lock file maintenance (#26205) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency typescript to v5.2.2 (#26390) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Michael Telatynski <[email protected]> * Correctly fill window.matrixChat even when a Wrapper module is active (#26395) Signed-off-by: Dominik Henneke <[email protected]> * Update react-types (#26124) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Add automation to move Element-R issues to crypto board (#26398) * triage-labelled: Fix workflow * Adds labs label to Element R issues (#26324) * triage-labelled: Fix step input names * triage-labelled: also set "Status" on ER issues * Revert "triage-labelled: also set "Status" on ER issues" This reverts commit e4d365a. This is better done through the project board automation, otherwise we risk moving items out of "In Progress" back to "Qualification". * docs: fixed multiple typos (#26403) * OIDC: document config (#26400) * document OIDC config * tidy * Delete .github/workflows/upgrade_dependencies.yml * Update triage-assigned.yml Update user name * Enable rust cryptography on develop.element.io (#26291) Once this lands, *existing* sessions will continue to use legacy crypto, but *new* logins will use rust crypto. * Upgrade matrix-react-sdk to 3.83.0 * Upgrade matrix-js-sdk to 29.1.0 * Prepare changelog for v1.11.47 * v1.11.47 * Reset matrix-react-sdk back to develop branch * Reset matrix-js-sdk back to develop branch * Document `feedback.new_issue_url` & `feedback.existing_issues_url` (#26446) * Update update-topics.yaml * Update update-topics.yaml * Iterate build_debian.yaml workflow (#26434) * Update config.md * Appease the linter * Add mdbook-mermaid (#26426) * Update docs.yml * Remove mermaid config section Signed-off-by: Michael Telatynski <[email protected]> * Add script to generate mermaid graph of automations (#26432) * Update docs.yml * Bump browserify-sign from 4.2.1 to 4.2.2 (#26459) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update docs.yml * Update docs.yml * Vendor in Mermaid v10 for frontmatter title support Signed-off-by: Michael Telatynski <[email protected]> * Fix path issues Signed-off-by: Michael Telatynski <[email protected]> * prettierignore Signed-off-by: Michael Telatynski <[email protected]> * Fix docs workflow Signed-off-by: Michael Telatynski <[email protected]> * Use olm from default npm registry, since it's there now (#26461) * Release prep script for cutting branches (#26392) * Update release_prepare.yml * Upgrade matrix-react-sdk to 3.84.0-rc.1 * Upgrade matrix-js-sdk to 30.0.0-rc.1 * Prepare changelog for v1.11.48-rc.1 * v1.11.48-rc.1 * Update titoportas/update-project-fields digest to 421a544 (#26478) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update definitelyTyped (#26479) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency @types/jest to v29.5.7 (#26480) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update actions/setup-node action to v4 (#26484) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency eslint-plugin-unicorn to v49 (#26485) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#26482) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update build_debian.yaml * Ship OpenCoDE variant to Dockerhub (#26399) * Make triage-labelled workflow reusable * Add input parameter for ELEMENT_BOT_TOKEN * Upgrade matrix-react-sdk to 3.84.0 * Upgrade matrix-js-sdk to 30.0.0 * Prepare changelog for v1.11.48 * v1.11.48 * Reset matrix-react-sdk back to develop branch * Reset matrix-js-sdk back to develop branch * Upgrade webpack to 5.89.0 * Remove unused extract-text-webpack-plugin * Replace optimize-css-assets-webpack-plugin with css-minimizer-webpack-plugin * Eliminate use of loader-utils * Appease the linter * Ship element-web as a debian package (#26533) * Update @matrix-org/react-sdk-module-api * Remove obsolete comment * Only show overlay on build errors * Eliminate worklet-loader * Appease the linter * Only update repo description once (#26517) * Update translating-dev.md * Run Localazy Download on a schedule 3 times a week instead of during release process (#26550) * Add defaults and fix comment * Disable automatic publicPath to unbreak worklets * Update CODEOWNERS * [create-pull-request] automated change * [Backport staging] Ship element-web as a debian package (#26563) Co-authored-by: Michael Telatynski <[email protected]> * Upgrade matrix-react-sdk to 3.84.1 * Upgrade matrix-js-sdk to 30.0.1 * Prepare changelog for v1.11.49 * v1.11.49 * Update build_debian.yaml * New release automations (#26495) * Squashed commit of the following: commit 069c1bc8331b460275caf492450436f5e127f767 Author: Johannes Marbach <[email protected]> Date: Sat Nov 11 16:08:30 2023 +0100 Replace worker-loader with built-in Webpack 5 support for web workers * Update mocks for workers / worklets * Update release-drafter.yml * Update build_develop.yml * Explicitly specify secrets Signed-off-by: Michael Telatynski <[email protected]> * Upgrade matrix-react-sdk to 3.85.0-rc.1 * Upgrade matrix-js-sdk to 30.1.0-rc.1 * v1.11.50-rc.0 * Use only chokidar for watching and add more logging * Appease the linter * Prevent copy-res -w from triggering unnecessary changes while webpack is already building * Remove unused dependencies and add transitive deps (#26543) * Update actions/github-script action to v7 (#26583) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#26581) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Bump crypto-js from 4.1.1 to 4.2.0 (#26460) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update definitelyTyped (#26579) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update babel monorepo to v7.23.3 (#26578) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Michael Telatynski <[email protected]> * Update dependency @types/jest to v29.5.8 (#26580) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Michael Telatynski <[email protected]> * Update build_debian.yaml * v1.11.50-rc.1 * Replace copy/watching of non-language assets with copy-webpack-plugin * Try explicitly relative path for config.json * Revert "Try explicitly relative path for config.json" This reverts commit 46ecafa. * Allow config.json to be missing * Update copy-webpack-plugin to latest * Undo auto public path disablement as it doesn't work with workers * Add feature_disable_call_per_sender_encryption documentation (in labs.md) (#26548) * add feature_disable_call_per_sender_encryption doc Co-authored-by: aceArt-GmbH <[email protected]> --------- Signed-off-by: Timo K <[email protected]> Co-authored-by: aceArt-GmbH <[email protected]> * Add custom loader for recorder worklet * Move worklet rule up to where it used to be * Set es2022 globally but force CommonJS when using ts-node for scripts * Add documentation around releases (#26602) * Upgrade matrix-react-sdk to 3.85.0 * Upgrade matrix-js-sdk to 30.1.0 * v1.11.50 * Update release-gitflow.yml * Reset matrix-react-sdk back to develop branch * Reset matrix-js-sdk back to develop branch * Update CHANGELOG.md * Set up rudimentary workflow for manually triggering label sync * Make it preeeeeetty * Use correct key * Rename action to not tap outside exclusion list * Add labels currently added with respect to element-meta * Fix quotes * Quote names * Prettier * Improve debian package and docs (#26618) * Setup CI to lint workflows with action-validator * Add Jitsi SDK as a static resource and set up CI to update it on change * Make prettier ignore jitsi_external_api.min.js * Improve rendering style of markdown in the mdbook (#26616) * Update path Signed-off-by: Michael Telatynski <[email protected]> * Add license and copy it into bundle while making terser ignore the SDK * Enable new room header by default on develop Relates to: element-hq/element-web#25883 * Adapt to renaming of release-action workflow * Improve Module API docs & example (#26637) * Fix types Signed-off-by: Michael Telatynski <[email protected]> * Upgrade matrix-react-sdk to 3.86.0-rc.2 * Upgrade matrix-js-sdk to 30.2.0-rc.0 * v1.11.51-rc.0 * Update definitelyTyped (#26666) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update docker/build-push-action digest to 4a13e50 (#26663) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update babel monorepo to v7.23.4 (#26665) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#26664) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency @types/jest to v29.5.10 (#26667) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency typescript to v5.3.2 (#26668) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * [create-pull-request] automated change * Adapt label-sync workflow to new version of upstream workflow * Add missing labels for label sync This adds all the labels that element-web has over element-meta to labels.yml. There is some deduplication to do that we can take care of over time. This at least should ensure that we don't lose anything we have to day while still benefiting from label sync. * Replace quotes * Pretty it up * Enable label sync workflow * Undry the workflow * Upgrade matrix-react-sdk to 3.86.0 * Upgrade matrix-js-sdk to 30.2.0 * v1.11.51 * Reset matrix-react-sdk back to develop branch * Reset matrix-js-sdk back to develop branch * Add @nordeck/[email protected] to variants/openDesk/build_config.yaml Signed-off-by: Charly Nguyen <[email protected]> * [create-pull-request] automated change * update to org to element-hq * Update org in scripts and workflows * Upgrade matrix-react-sdk to 3.87.0-rc.0 * Upgrade matrix-js-sdk to 30.3.0-rc.0 * v1.11.52-rc.0 * Upgrade matrix-react-sdk to 3.87.0 * Upgrade matrix-js-sdk to 30.3.0 * v1.11.52 * Upgrade matrix-react-sdk to 3.88.0 * v1.11.53 * Remove useless patch better-text-for-tab-switch * Upgrade patches * Upgrade conflicting patches * Upgrade conflict patch bug-reporting. Remove two files from the patch, not needed any more. * Upgrade conflicted patch customize-room-header-bar * Upgrade conflicted patch export-room-members * Upgrade more conflicting patches * Remove patch remove-fdroid (bug is fixed) * Remove better-text-for-tab-switch from patches.json * Fix conflicted patch show-icon-on-public-room * Last patch fix * Translation fixes + an updated getVersionInfo for the rust crypto version. * Fix customize-room-header-bar : messed up brackets * New style for Export room members button * Remove no-voip-mentions-in-onboarding patch. Replace with custom translation and customisations-replaced image. * Fix DecoratedRoomAvatar display bug #890 * Fix one last translation * Update cypress tests * Delete .github/labels.yml we don't neeed extra labels * fix use of MatrixClientContext --------- Signed-off-by: Dominik Henneke <[email protected]> Signed-off-by: Michael Telatynski <[email protected]> Signed-off-by: Timo K <[email protected]> Signed-off-by: Charly Nguyen <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Michael Telatynski <[email protected]> Co-authored-by: Dominik Henneke <[email protected]> Co-authored-by: Richard van der Hoff <[email protected]> Co-authored-by: Richard van der Hoff <[email protected]> Co-authored-by: Germain <[email protected]> Co-authored-by: Subash-Lamichhane <[email protected]> Co-authored-by: Kerry <[email protected]> Co-authored-by: Johannes Marbach <[email protected]> Co-authored-by: ElementRobot <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Hubert Chathi <[email protected]> Co-authored-by: Charly Nguyen <[email protected]> Co-authored-by: t3chguy <[email protected]> Co-authored-by: Timo <[email protected]> Co-authored-by: aceArt-GmbH <[email protected]> Co-authored-by: David Langley <[email protected]> Co-authored-by: Florian Duros <[email protected]> Co-authored-by: Olivier D <[email protected]>
I'm not sure if this is the right place to mention it, but it would be nice if the items from right panel would also open in the right panel - at least these that could easily fit in. People, files and pinned are already there, so why not move also "Polls", "Export chat" and "Copy link" to the right panel? I am not sure about "Invite" (maybe could also?), and "Settings" would really not fit, but the theree mentioned should fit without problems. |
When using the new room header, the "Layout" option when inside an element call is missing. Because of this there is no option to increase the size of another screen share when sharing the screen yourself. (My own screen share gets put into spotlight and there is no way to switch to "Freedom Layout" because the button is missing in the new room header.) I know I can maximize the other screen share but then element is in full screen, this is sub-optimal. |
feedback on the room header that the new height and moving icon makes it hard to use the window |
|
Cmd+F is opt-in on Electron, on Web it is the browser's search instead.
Not sure what this is asking |
@t3chguy I believe you're working on and have completed the following item:
If you have not completed this work, feel free to create it as a task so it can be tracked and the AC written. |
Moving the issue into "Q3" on the web roadmap as it's not yet released to the public. |
@daniellekirkwood sorry I am not sure what that task is. It doesn't have any detail to explain itself. If it is the RoomSummaryCard stuff then this is currently in review: matrix-org/matrix-react-sdk#12793 |
that PR doesn't have enough detail in it for me to know what it is @t3chguy |
We're nearly out of Labs on this topic. Once released, we can close this issue. There will be a follow up issue to track the remaining iterations on this topic. |
Just encountered the refreshed sidebar in Element Desktop. Where is the close button in the top level view? There's just the 3 tabs: Info, People, Threads. There's a close button when you're inside a thread, or profile, or Files under Info, but it's gone from the overview. Should I open a new bug? |
@opusforlife2 no you should use the existing one: #27799 |
Ah. Thanks. Kept searching for sidebar. |
I miss global search, how do I access this? |
@daniellekirkwood All Rooms search has moved somewhat, the search field is now in the right panel |
All items have been answered or moved. The Labs flag is being removed from product. There are more iterations to be done, these are captured in another issue so this one can be closed when that PR lands I think @t3chguy ... |
Design: Figma document Trust & Decoration
Updated figma May 2024: https://www.figma.com/design/gpxHFDTNK796n84r1ZxYG2/Room-Header-%26-Details-(2024)?node-id=0-1
Designer: @americanrefugee
PM: @daniellekirkwood / @nadonomy
Developer: @germain-gg
Room header
feature_state_counters
vector-im/element-web#25884develop.element.io
vector-im/element-web#26089Room details
Open Questions
feature_right_panel_default_open
feature_state_counters
: Deprecatefeature_state_counters
#25884Time sheeting ⏱️
WEB: Room header & details
The text was updated successfully, but these errors were encountered: