Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Commit

Permalink
De-labs Metaspaces (#7613)
Browse files Browse the repository at this point in the history
  • Loading branch information
t3chguy authored Jan 25, 2022
1 parent ecd5be8 commit 8ddd677
Show file tree
Hide file tree
Showing 8 changed files with 11 additions and 45 deletions.
4 changes: 4 additions & 0 deletions res/css/views/settings/tabs/user/_SidebarUserSettingsTab.scss
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ limitations under the License.
*/

.mx_SidebarUserSettingsTab {
.mx_SettingsTab_section {
margin-top: 10px;
}

.mx_SidebarUserSettingsTab_subheading {
font-size: $font-15px;
line-height: $font-24px;
Expand Down
15 changes: 6 additions & 9 deletions src/components/views/dialogs/UserSettingsDialog.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -127,15 +127,12 @@ export default class UserSettingsDialog extends React.Component<IProps, IState>
"mx_UserSettingsDialog_keyboardIcon",
<KeyboardUserSettingsTab />,
));

if (SettingsStore.getValue("feature_spaces_metaspaces")) {
tabs.push(new Tab(
UserTab.Sidebar,
_td("Sidebar"),
"mx_UserSettingsDialog_sidebarIcon",
<SidebarUserSettingsTab />,
));
}
tabs.push(new Tab(
UserTab.Sidebar,
_td("Sidebar"),
"mx_UserSettingsDialog_sidebarIcon",
<SidebarUserSettingsTab />,
));

if (SettingsStore.getValue(UIFeature.Voip)) {
tabs.push(new Tab(
Expand Down
3 changes: 1 addition & 2 deletions src/components/views/spaces/SpacePanel.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,6 @@ const InnerSpacePanel = React.memo<IInnerSpacePanelProps>(({ children, isPanelCo
});

const SpacePanel = () => {
const metaSpacesEnabled = useSettingValue("feature_spaces_metaspaces");
const [isPanelCollapsed, setPanelCollapsed] = useState(true);
const ref = useRef<HTMLUListElement>();
useLayoutEffect(() => {
Expand Down Expand Up @@ -366,7 +365,7 @@ const SpacePanel = () => {
) }
</Droppable>

{ metaSpacesEnabled && <QuickSettingsButton isPanelCollapsed={isPanelCollapsed} /> }
<QuickSettingsButton isPanelCollapsed={isPanelCollapsed} />
</ul>
) }
</RovingTabIndexProvider>
Expand Down
1 change: 0 additions & 1 deletion src/i18n/strings/en_EN.json
Original file line number Diff line number Diff line change
Expand Up @@ -885,7 +885,6 @@
"Show extensible event representation of events": "Show extensible event representation of events",
"Location sharing (under active development)": "Location sharing (under active development)",
"Show info about bridges in room settings": "Show info about bridges in room settings",
"Meta Spaces": "Meta Spaces",
"Use new room breadcrumbs": "Use new room breadcrumbs",
"New spotlight search experience": "New spotlight search experience",
"Right panel stays open (defaults to room member list)": "Right panel stays open (defaults to room member list)",
Expand Down
14 changes: 0 additions & 14 deletions src/settings/Settings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -332,17 +332,6 @@ export const SETTINGS: {[setting: string]: ISetting} = {
displayName: _td("Show info about bridges in room settings"),
default: false,
},
"feature_spaces_metaspaces": {
isFeature: true,
labsGroup: LabGroup.Spaces,
supportedLevels: LEVELS_FEATURE,
displayName: _td("Meta Spaces"),
default: false,
controller: new OrderedMultiController([
new IncompatibleController("showCommunitiesInsteadOfSpaces"),
new ReloadOnChangeController(),
]),
},
"feature_breadcrumbs_v2": {
isFeature: true,
labsGroup: LabGroup.Rooms,
Expand Down Expand Up @@ -863,9 +852,6 @@ export const SETTINGS: {[setting: string]: ISetting} = {
default: {
[MetaSpace.Home]: true,
},
controller: new IncompatibleController("feature_spaces_metaspaces", {
[MetaSpace.Home]: true,
}, false),
},
"Spaces.showPeopleInSpace": {
supportedLevels: [SettingLevel.ROOM_ACCOUNT],
Expand Down
1 change: 0 additions & 1 deletion test/stores/SpaceStore-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ limitations under the License.
import { EventType } from "matrix-js-sdk/src/@types/event";
import { RoomMember } from "matrix-js-sdk/src/models/room-member";

import "./enable-metaspaces-labs";
import "../skinned-sdk"; // Must be first for skinning to work
import SpaceStore from "../../src/stores/spaces/SpaceStore";
import {
Expand Down
17 changes: 0 additions & 17 deletions test/stores/enable-metaspaces-labs.ts

This file was deleted.

1 change: 0 additions & 1 deletion test/stores/room-list/SpaceWatcher-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

import "../enable-metaspaces-labs";
import "../../skinned-sdk"; // Must be first for skinning to work
import { SpaceWatcher } from "../../../src/stores/room-list/SpaceWatcher";
import type { RoomListStoreClass } from "../../../src/stores/room-list/RoomListStore";
Expand Down

0 comments on commit 8ddd677

Please sign in to comment.