Skip to content

Conversation

@snuglinux
Copy link

Hiding buttons creating home, creating room, creating space and hiding export item.
Example in config.json:

    "setting_defaults": {
        "RustCrypto.staged_rollout_percent": 60,
        "MessageComposerInput.showStickersButton": false,
        "audioNotificationsEnabled": true,
        "view_button_creating_space": false,
        "view_button_creating_room": false,
        "view_button_creating_home": false,
        "view_button_export": false
    },

@snuglinux snuglinux requested a review from a team as a code owner April 29, 2025 14:56
@snuglinux snuglinux requested review from dbkr and florianduros April 29, 2025 14:56
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Apr 29, 2025
@t3chguy t3chguy requested a review from a team April 29, 2025 14:57
/>
)}

{/*Pro777 edit*/ !isVideoRoom && SettingsStore.getValue("view_button_export") && (
Copy link
Member

Choose a reason for hiding this comment

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

These comments seem unrelated to this project

Copy link
Member

Choose a reason for hiding this comment

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

Unrelated file

Comment on lines +359 to +360
// Pro777 edit
///if (showCreateRoom || showExploreRooms) {
Copy link
Member

Choose a reason for hiding this comment

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

Unrelated comment & commented out code

Copy link
Member

Choose a reason for hiding this comment

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

Unrelated file

Comment on lines +295 to +298
"view_button_creating_space": IBaseSetting<boolean>;
"view_button_creating_room": IBaseSetting<boolean>;
"view_button_creating_home": IBaseSetting<boolean>;
"view_button_export": IBaseSetting<boolean>;
Copy link
Member

Choose a reason for hiding this comment

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

Should use the UIFeature. nomenclature

{contextMenu}
</li>
);
// Pro777 edit
Copy link
Member

Choose a reason for hiding this comment

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

Same as earlier

}
}

// Pro777 edit
Copy link
Member

Choose a reason for hiding this comment

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

Same as earlier

import SpaceContextMenu from "../context_menus/SpaceContextMenu";
import InlineSpinner from "../elements/InlineSpinner";
import { HomeButtonContextMenu } from "../spaces/SpacePanel";
// Pro777 edit
Copy link
Member

Choose a reason for hiding this comment

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

Same as earlier

Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

The title says in the configuration file but then creates account settings, please align the title with the implementation.

@t3chguy
Copy link
Member

t3chguy commented Jun 23, 2025

Closing due to no iteration based on review.

@t3chguy t3chguy closed this Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants