Fixes issue with file/line pinning getting lost

Author: eamodio

CHANGELOG.md

@@ -49,6 +49,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/) and this p
 - Fixes an issue where many views wouldn't refresh properly when going from no items to some items
 - Fixes an issue where _Publish Branch_ was incorrectly showing up on remote branches
 - Fixes an issue where the _Open Directory Compare \*_ commands failed to work
+- Fixes an issue where pinning a file/line to the _File History_ view or _Line History_ view would get lost if the view was collapsed and expanded
 
 ### Removed
 

src/views/lineHistoryView.ts

@@ -13,6 +13,8 @@ export class LineHistoryView extends ViewBase<LineHistoryTrackerNode, LineHistor
 
 	constructor() {
 		super('gitlens.views.lineHistory', 'Line History');
+
+		void setContext(ContextKeys.ViewsLineHistoryEditorFollowing, true);
 	}
 
 	protected get showCollapseAll(): boolean {
@@ -69,16 +71,25 @@ export class LineHistoryView extends ViewBase<LineHistoryTrackerNode, LineHistor
 	}
 
 	private setEditorFollowing(enabled: boolean) {
+		const root = this.ensureRoot();
+		if (!root.hasUri) return;
+
 		void setContext(ContextKeys.ViewsLineHistoryEditorFollowing, enabled);
+
 		this.root?.setEditorFollowing(enabled);
 
 		if (this.description?.endsWith(pinnedSuffix)) {
 			if (enabled) {
 				this.description = this.description.substr(0, this.description.length - pinnedSuffix.length);
 			}
-		} else if (!enabled) {
+		} else if (!enabled && this.description != null) {
 			this.description += pinnedSuffix;
 		}
+
+		if (enabled) {
+			void root.ensureSubscription();
+			void this.refresh(true);
+		}
 	}
 
 	private setShowAvatars(enabled: boolean) {

src/views/nodes/fileHistoryTrackerNode.ts

@@ -119,6 +119,8 @@ export class FileHistoryTrackerNode extends SubscribeableViewNode<FileHistoryVie
 	async refresh(reset: boolean = false) {
 		const cc = Logger.getCorrelationContext();
 
+		if (!this.canSubscribe) return false;
+
 		if (reset) {
 			this.setUri();
 			this.resetChild();

src/views/nodes/lineHistoryTrackerNode.ts

@@ -125,6 +125,8 @@ export class LineHistoryTrackerNode extends SubscribeableViewNode<FileHistoryVie
 	async refresh(reset: boolean = false) {
 		const cc = Logger.getCorrelationContext();
 
+		if (!this.canSubscribe) return false;
+
 		if (reset) {
 			this.setUri();
 			this._editorContents = undefined;