Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Style-preview: Add scroll buttons #11039

Merged

Conversation

Darshan-upadhyay1110
Copy link
Contributor

  • added scroll up and down btn for stylepreview in notebookbar
  • added 'more' button option which is a toggle button opens the stylist

Change-Id: I230ddefe3cb68dad3ba46b12eaabb12e99bad6db

Summary

TODO

  • ...

Checklist

  • I have run make prettier-write and formatted the code.
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

@Darshan-upadhyay1110
Copy link
Contributor Author

#10998 is partially done

Copy link
Contributor

@eszkadev eszkadev left a comment

Choose a reason for hiding this comment

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

really nice - no custom iconview widget :)

@eszkadev
Copy link
Contributor

It should go in together with: #11040
(make style sidebar non-experimental)

@Darshan-upadhyay1110 Darshan-upadhyay1110 force-pushed the private/darshan/add-scroll-btns-in-stylepreview branch from a4490ad to 75df47c Compare January 29, 2025 11:32
@Darshan-upadhyay1110 Darshan-upadhyay1110 force-pushed the private/darshan/add-scroll-btns-in-stylepreview branch from 75df47c to 1b80927 Compare January 30, 2025 06:47
Copy link
Contributor

@pedropintosilva pedropintosilva left a comment

Choose a reason for hiding this comment

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

it works, Thanks!

Also please use more minimal buttons for the scroll up and down (example: search up and down buttons)

browser/css/notebookbar.css Outdated Show resolved Hide resolved
browser/css/notebookbar.css Outdated Show resolved Hide resolved
browser/css/notebookbar.css Outdated Show resolved Hide resolved
@Darshan-upadhyay1110 Darshan-upadhyay1110 force-pushed the private/darshan/add-scroll-btns-in-stylepreview branch from 1b80927 to 4ba2c13 Compare February 3, 2025 07:06
- added scroll up and down btn for stylepreview in notebookbar
- added 'more' button option which is a toggle button opens the stylist

Signed-off-by: Darshan-upadhyay1110 <[email protected]>
Change-Id: I230ddefe3cb68dad3ba46b12eaabb12e99bad6db
@Darshan-upadhyay1110 Darshan-upadhyay1110 force-pushed the private/darshan/add-scroll-btns-in-stylepreview branch from 4ba2c13 to 6eb9fe3 Compare February 3, 2025 07:15
@pedropintosilva
Copy link
Contributor

Should this wait for RED window?

@eszkadev
Copy link
Contributor

eszkadev commented Feb 3, 2025

I think it is fine form code perspective (not much inpact on other things)

@pedropintosilva pedropintosilva merged commit 57e28ad into master Feb 3, 2025
14 checks passed
@pedropintosilva pedropintosilva deleted the private/darshan/add-scroll-btns-in-stylepreview branch February 3, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Deeper Style UI: Run in dialog not sidebar
3 participants