Skip to content

Commit 2f49e8d

Browse files
committed
feat: remove default image gallery selected state
1 parent 3450c49 commit 2f49e8d

1 file changed

Lines changed: 0 additions & 29 deletions

File tree

src/LiveDevelopment/main.js

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@ define(function main(require, exports, module) {
3939
LivePreviewTransport = require("LiveDevelopment/MultiBrowserImpl/transports/LivePreviewTransport"),
4040
CommandManager = require("command/CommandManager"),
4141
PreferencesManager = require("preferences/PreferencesManager"),
42-
StateManager = require("preferences/StateManager"),
4342
UrlParams = require("utils/UrlParams").UrlParams,
4443
Strings = require("strings"),
4544
ExtensionUtils = require("utils/ExtensionUtils"),
@@ -57,10 +56,6 @@ define(function main(require, exports, module) {
5756

5857
const PREFERENCE_LIVE_PREVIEW_MODE = CONSTANTS.PREFERENCE_LIVE_PREVIEW_MODE;
5958

60-
// state manager key to track image gallery selected state, by default we keep this as selected
61-
// if this is true we show the image gallery when an image element is clicked
62-
const IMAGE_GALLERY_STATE = "livePreview.imageGallery.state";
63-
6459
PreferencesManager.definePreference(PREFERENCE_LIVE_PREVIEW_MODE, "string", LIVE_HIGHLIGHT_MODE, {
6560
description: StringUtils.format(
6661
Strings.LIVE_PREVIEW_MODE_PREFERENCE, LIVE_PREVIEW_MODE, LIVE_HIGHLIGHT_MODE, LIVE_EDIT_MODE),
@@ -70,34 +65,11 @@ define(function main(require, exports, module) {
7065
_previewModeUpdated();
7166
});
7267

73-
/**
74-
* get the image gallery state from StateManager
75-
* @returns {boolean} true (default)
76-
*/
77-
function _getImageGalleryState() {
78-
const savedState = StateManager.get(IMAGE_GALLERY_STATE);
79-
return savedState !== undefined && savedState !== null ? savedState : true;
80-
}
81-
82-
/**
83-
* sets the image gallery state
84-
* @param {Boolean} the state that we need to set
85-
*/
86-
function setImageGalleryState(state) {
87-
StateManager.set(IMAGE_GALLERY_STATE, state);
88-
89-
// update the config with the new state
90-
const config = MultiBrowserLiveDev.getConfig();
91-
config.imageGalleryState = state;
92-
MultiBrowserLiveDev.updateConfig(config);
93-
}
94-
9568
let params = new UrlParams();
9669
const defaultConfig = {
9770
mode: LIVE_HIGHLIGHT_MODE, // will be updated when we fetch entitlements
9871
elemHighlights: CONSTANTS.HIGHLIGHT_HOVER, // default value, this will get updated when the extension loads
9972
showRulerLines: false, // default value, this will get updated when the extension loads
100-
imageGalleryState: _getImageGalleryState(), // image gallery selected state
10173
isPaidUser: false, // will be updated when we fetch entitlements
10274
isLoggedIn: false, // will be updated when we fetch entitlements
10375
hasLiveEditCapability: false // handled inside _liveEditCapabilityChanged function
@@ -373,7 +345,6 @@ define(function main(require, exports, module) {
373345
exports.isActive = isActive;
374346
exports.setLivePreviewPinned = setLivePreviewPinned;
375347
exports.setLivePreviewTransportBridge = setLivePreviewTransportBridge;
376-
exports.setImageGalleryState = setImageGalleryState;
377348
exports.updateElementHighlightConfig = updateElementHighlightConfig;
378349
exports.updateRulerLinesConfig = updateRulerLinesConfig;
379350
exports.getConnectionIds = MultiBrowserLiveDev.getConnectionIds;

0 commit comments

Comments
 (0)