Skip to content

Conversation

@myjeong19
Copy link
Member

@myjeong19 myjeong19 commented Oct 1, 2024

Description of Changes

Here is reference: #25


support 'custom your created image' block

-> add custom solution for rendering 'image'

Review point

To reproduce

  • npm run story:start
  • click 'image' on the Storybook left side panel

Screenshot

  • A screenshot of your created block being rendered in Storybook.
E1.84.92.E1.85.AA.E1.84.86.E1.85.A7.E1.86.AB.20.E1.84.80.E1.85.B5.E1.84.85.E1.85.A9.E1.86.A8.202024-10-02.20.E1.84.8B.E1.85.A9.E1.84.8C.E1.85.A5.E1.86.AB.201.mp4

Review Guide

Reviews are conducted based on priority levels, such as p0, p1, p2, p3, p4, and p5.
p0 ~ p2: If the author decides not to reflect a review for p0 to p2, it signals that a proper discussion with the reviewer is
necessary. It is expected that the review will be resolved either through incorporating the feedback or through further discussion.
p3: indicates that the reviewer has identified a significant issue, but either lacks a clear solution or the comment lacks sufficient context. Further explanation or additional discussion on the reviewer's concerns is needed.
p4, p5: p4 and p5 suggest low priority, and if the author does not deem them important, these comments can be disregarded.

@myjeong19 myjeong19 added the draft it is draft pr label Oct 1, 2024
@myjeong19 myjeong19 changed the title draft. image block Feat. image block Oct 1, 2024
@myjeong19 myjeong19 added enhancement New feature or request and removed draft it is draft pr labels Oct 1, 2024
hint={`${currentImageIndex + 2} of ${imageLength} `}
disabled={!hasNext}
>
<motion.button
Copy link
Collaborator

Choose a reason for hiding this comment

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

p2: I think motion.button is not neccessary. you can use just button


<div className="notion-image-viewer-scaler">
<Tooltip content="Zoom out" hint="-">
<motion.button
Copy link
Collaborator

Choose a reason for hiding this comment

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

p2: I think motion.button is not neccessary. you can use just button

<span>%</span>
</div>
) : (
<motion.button
Copy link
Collaborator

Choose a reason for hiding this comment

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

p2: I think motion.button is not neccessary. you can use just button

@anonymousRecords anonymousRecords linked an issue Oct 7, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Notion Block] Support Image notion block

3 participants