Please address the comments from this code review:

## Overall Comments
- Extract the repeated dp-based size calculations (e.g. dp(8), dp(48), dp(47), dp(12)) into named constants or a helper method instead of using magic numbers scattered throughout.
- Unify the orientation check logic (mixing parentWidth/parentHeight and videoWidth/videoHeight) into a single helper or state to avoid subtle inconsistencies and duplicated conditionals.
- Remove or replace the "Need further adjust" inline comments by finalizing the seekbar sizing logic or linking to a tracked issue so that the code doesn’t ship with unresolved FIXMEs.

## Individual Comments

### Comment 1
<location> `TMessagesProj/src/main/java/org/telegram/ui/PhotoViewer.java:3504` </location>
<code_context>
                 if (exitFullscreenButton.getVisibility() != VISIBLE) {
                     exitFullscreenButton.setVisibility(VISIBLE);
                 }
+                exitFullscreenButton.setImageResource(parentWidth > parentHeight ? R.drawable.msg_minvideo : R.drawable.msg_maxvideo);
                 extraWidth = dp(48);
                 layoutParams.rightMargin = dp(47);
</code_context>

<issue_to_address>
**question:** Image resource selection logic may be unclear for edge cases.

If parent and video aspect ratios can differ, please clarify which should determine the icon, or add documentation for the intended behavior in such cases.
</issue_to_address>

### Comment 2
<location> `TMessagesProj/src/main/java/org/telegram/ui/PhotoViewer.java:3542` </location>
<code_context>
                 timeSpring.start();
             } else {
-                videoPlayerSeekbar.setSize(getMeasuredWidth() - dp(2 + 14) - size - extraWidth, getMeasuredHeight());
+                videoPlayerSeekbar.setSize(getMeasuredWidth() - dp(8) - size - extraWidth, getMeasuredHeight()); //Need further adjust
                 timeValue.setValue(size);
             }
</code_context>

<issue_to_address>
**suggestion:** Repeated use of dp(8) for seekbar sizing may need centralization.

Consider defining dp(8) as a constant or utility to maintain consistency and simplify future changes.

Suggested implementation:

```java
                if (exitFullscreenButton.getVisibility() != GONE) { //see the video I sent in the group, I initially suspected the button wasn't fully hidden, so I made this change. Revert if necessary.
                    exitFullscreenButton.setVisibility(GONE);
                }
                extraWidth = 0;
                layoutParams.rightMargin = dp(12);
                timeSpring.getSpring().setFinalPosition(size);
                timeSpring.start();
            } else {

```

```java
                    videoPlayerSeekbar.setSize((int) (getMeasuredWidth() - dp(SEEKBAR_HORIZONTAL_PADDING) - value - extraWidth), getMeasuredHeight()); //Need further adjust
                });

```

```java
        // Centralized seekbar horizontal padding for sizing
        private static final int SEEKBAR_HORIZONTAL_PADDING = 8;

        public VideoPlayerControlFrameLayout(@NonNull Context context) {
            int extraWidth;
            ignoreLayout = true;
            LayoutParams layoutParams = (LayoutParams) videoPlayerTime.getLayoutParams();

```

If there are other usages of `dp(8)` for seekbar sizing elsewhere in the file, they should also be replaced with `dp(SEEKBAR_HORIZONTAL_PADDING)` for consistency.
</issue_to_address>

### Comment 3
<location> `TMessagesProj/src/main/java/org/telegram/ui/PhotoViewer.java:9908-9910` </location>
<code_context>
+            if (AndroidUtilities.displaySize.x > AndroidUtilities.displaySize.y) {
+                parentActivity.setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_PORTRAIT);
+            } else {
+                WindowManager manager = (WindowManager) parentActivity.getSystemService(Activity.WINDOW_SERVICE);
+                int displayRotation = manager.getDefaultDisplay().getRotation();
+                parentActivity.setRequestedOrientation(displayRotation == Surface.ROTATION_270 ?
+                    ActivityInfo.SCREEN_ORIENTATION_REVERSE_LANDSCAPE : ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE);
+                toggleActionBar(false, false);
</code_context>

<issue_to_address>
**question:** Orientation logic may not handle all rotation values.

Explicitly handle ROTATION_0 and ROTATION_90, or clarify why only ROTATION_270 is singled out.
</issue_to_address>

### Comment 4
<location> `TMessagesProj/src/main/java/org/telegram/ui/PhotoViewer.java:11008` </location>
<code_context>
-                if (fullscreenButton[b].getVisibility() != View.GONE) {
-                    fullscreenButton[b].setVisibility(View.GONE);
-                }
+            if (AndroidUtilities.displaySize.y > AndroidUtilities.displaySize.x && w > h && b == 0) {
+                videoWidth = w;
+                videoHeight = h;
</code_context>

<issue_to_address>
**question:** videoWidth and videoHeight assignment is limited to b == 0.

Please verify that only the first fullscreen button should update videoWidth and videoHeight, and that this behavior is correct for all use cases.
</issue_to_address>