Skip to content

Add responsive UI#2187

Open
mattnotmitt wants to merge 194 commits intogchq:masterfrom
mattnotmitt:181-responsive-ui
Open

Add responsive UI#2187
mattnotmitt wants to merge 194 commits intogchq:masterfrom
mattnotmitt:181-responsive-ui

Conversation

@mattnotmitt
Copy link
Copy Markdown
Collaborator

@mattnotmitt mattnotmitt commented Feb 9, 2026

Taking over from @valdelaseras' heroic efforts in 2023 to get this done, I'm going to support landing this. See the original PR #1615 for most discussion, the only "original" work here was resolving merge conflicts and some small bug fixes.

My changes:

  • Operation counts
  • Allow toggling on favourite icon
  • Notify users that a single touch-tap does nothing in the operations list
  • Fix odd checkbox label position on macos
  • Fix odd label pinning
  • History update now happens on drag operation removal
  • Show compile message rather than "download cyberchef" in banner in mobile mode

Could be a nice base for CyberChef v11?

…ly revert some things once the mobile UI is solid, then patch up desktop view to its original state
…ate places, remove redundant css, update template to work with mobile UI )
… file, add new css files to split up huge IO stylesheet for better future DevX
…ate, so it is not visible anymore during load. Update TODO.md accordingly
a3957273 and others added 10 commits February 3, 2024 01:39
… 40), as per the current width implementations on init
- Operation counts
- Allow toggling on favourite icon
- Notify users that a single tap does nothing in the operations list
- Fix odd checkbox label position on macos
- Fix odd label pinning
- History update now happens on drag operation removal
- Show compile message rather than "download cyberchef" in banner
@GCHQDeveloper581 GCHQDeveloper581 linked an issue Feb 11, 2026 that may be closed by this pull request
@GCHQDeveloper581
Copy link
Copy Markdown
Contributor

Review request noted. It's a massive PR (for understandable reasons) so it won't be quick.

However I do note that @a3957273 had basically said "yes" apart from merge conflicts to the earlier PR #1615 so hopefully we can focus on your last few commits.

@GCHQDeveloper581 GCHQDeveloper581 added this to the v11 milestone Mar 20, 2026
@GCHQDeveloper581
Copy link
Copy Markdown
Contributor

GCHQDeveloper581 commented Mar 20, 2026

Not making any promises here, but have targeted this against the v11 milestone so that we can ensure it is considered amongst the planned changes.

@GCHQDeveloper581 GCHQDeveloper581 modified the milestones: v11, v12 Apr 18, 2026
@GCHQDeveloper581
Copy link
Copy Markdown
Contributor

Re-targeting at v12 as getting the v11 release (with Node update) out is becoming increasingly urgent.

The hope is that we can then start looking towards an early v12 release with this as a major component.

@mattnotmitt
Copy link
Copy Markdown
Collaborator Author

FYI @C85297 - I think the HTMLOperation.mjs file snuck its way in when you were resolving merge conflicts, just needs removing.
I've just given your merge commit a test and the search accessibility seems to be working properly with the new changes :)

@C85297
Copy link
Copy Markdown
Member

C85297 commented Apr 30, 2026

@mattnotmitt Oops good eye! Think it should be fine to leave since it has no diff against the master branch?

@mattnotmitt
Copy link
Copy Markdown
Collaborator Author

Yes, it's just not on the codepath anymore. The new web components replace it :)

@C85297
Copy link
Copy Markdown
Member

C85297 commented Apr 30, 2026

Oh right, got you. Fixed now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Misc: Mobile UI

5 participants