Skip to content

Tuning switching tabs interactions#19818

Open
FedeLoch wants to merge 10 commits into
pharo-project:Pharo14from
FedeLoch:tuning-switching-tabs-interactions
Open

Tuning switching tabs interactions#19818
FedeLoch wants to merge 10 commits into
pharo-project:Pharo14from
FedeLoch:tuning-switching-tabs-interactions

Conversation

@FedeLoch

@FedeLoch FedeLoch commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Tuning Alt tab + alt shift tab feature

Following #19776, in this pr, with @tesonep, we improved the UX by extending the feature allowing left and right arrows and...

We fixed the GC problem by caching the thumbs and only drawing them once per feature opening.

We profiled it by doing:

tabs := 30 timesCollect: [ :i | ClyFullBrowserMorph openOnClass: Object ].
AndreasSystemProfiler spyOn: [
		time := [
			        TaskListMorph allInstances first
				        openInWorld;
				        selectNextTask;
				        selectNextTask;
				        selectNextTask ] timeToRun ].
tabs do: #close.
time

Before GC behavior

Screenshot 2026-06-26 at 12 19 17

After GC behavior

Screenshot 2026-06-26 at 12 20 13

@request-info

request-info Bot commented Jun 26, 2026

Copy link
Copy Markdown

This issue has either a default title or empty body. We would appreciate it if you could provide more information. Note: I am not a very intelligent bot, I can only react to new comments. Please add a comment for me if you update the body or title.

Comment thread src/Morphic-Widgets-Taskbar/TaskbarTask.class.st Outdated
Comment thread src/Morphic-Widgets-Taskbar/TaskbarTask.class.st Outdated
Comment thread src/Morphic-Widgets-Taskbar/TaskListMorph.class.st
@jecisc

jecisc commented Jun 26, 2026

Copy link
Copy Markdown
Member

The CI does not work :(

Comment thread src/Kernel/Integer.class.st Outdated

^ (1 to: self) collect: [ :anIndex | aBlock cull: anIndex ]
]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure this should be commited here

@tesonep tesonep closed this Jun 30, 2026
@tesonep tesonep reopened this Jun 30, 2026
FedeLoch and others added 2 commits July 6, 2026 10:18
This method is not used in the current PR and needs more discussion to know if we integrate it or not. 

This deserves its own PR I think
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants