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.

Sign up for GitHub

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

Jump to bottom

🐛 修复课表详情教室与教师文字长度过长显示异常 #454

Merged
merged 1 commit into from
Dec 11, 2023
Merged

🐛 修复课表详情教室与教师文字长度过长显示异常 #454

merged 1 commit into from
Dec 11, 2023

Conversation

RQ527
Copy link
Contributor

@RQ527 RQ527 commented Dec 7, 2023

No description provided.

@985892345
Copy link
Member

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

@RQ527
Copy link
Contributor Author

RQ527 commented Dec 7, 2023

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

image

@RQ527
Copy link
Contributor Author

RQ527 commented Dec 7, 2023

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

image
image
@985892345

@RQ527
Copy link
Contributor Author

RQ527 commented Dec 7, 2023

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

@985892345
Copy link
Member

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

@RQ527
Copy link
Contributor Author

RQ527 commented Dec 7, 2023

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

不是，教师那个textview直接被挤出去了

@985892345
Copy link
Member

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

了解了，这里教室不带有跑马灯的效果，但改成两行应该是不好改的，因为后面教师是分开的。还有你的改法只是设置外层布局的maxWidth，你测试过你的改法有效吗，辛苦RQ看下哈 @RQ527

@RQ527
Copy link
Contributor Author

RQ527 commented Dec 7, 2023

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

了解了，这里教室不带有跑马灯的效果，但改成两行应该是不好改的，因为后面教师是分开的。还有你的改法只是设置外层布局的maxWidth，你测试过你的改法有效吗，辛苦RQ看下哈 @RQ527

我测试过了，有效的。我改的不是外层布局，改的是mTvClassroom的maxWidth

@985892345
Copy link
Member

985892345 commented Dec 7, 2023 •
edited
Loading

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

不是，教师那个textview直接被挤出去了

是不是应该设置成教室有一个maxWidth，然后跑马灯，然后老师刚好靠边不挤出更好？

@RQ527
Copy link
Contributor Author

RQ527 commented Dec 7, 2023

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

不是，教师那个textview直接被挤出去了

是不是应该设置成教室有一个maxWidth，然后跑马灯，然后老师刚好靠边不挤出更好？

老师的文字长度不能确定

@985892345
Copy link
Member

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

不是，教师那个textview直接被挤出去了

是不是应该设置成教室有一个maxWidth，然后跑马灯，然后老师刚好靠边不挤出更好？

老师的文字长度不能确定

为什么不能确定？

@RQ527
Copy link
Contributor Author

RQ527 commented Dec 7, 2023

不清楚这个显示异常是怎样的，辛苦RQ提供截图说明一下～ @RQ527 R

麻烦985大佬看一下有没有问题 [送花]

默认的应该就是跑马灯效果吧？

不是，教师那个textview直接被挤出去了

是不是应该设置成教室有一个maxWidth，然后跑马灯，然后老师刚好靠边不挤出更好？

老师的文字长度不能确定

为什么不能确定？

老师的长度不是固定值

985892345
985892345 approved these changes Dec 11, 2023
View reviewed changes
.../main/java/com/mredrock/cyxbs/course/page/course/ui/dialog/adapter/viewholder/StuLessonVH.kt Outdated
@@ -44,6 +48,15 @@ class StuLessonVH(
mTvTitle.text = data.course.course
mTvClassroom.text = data.course.classroom
mTvTeacher.text = data.course.teacher
//教室和教师文字内容加起来超过外层布局宽度时设置教室最大宽度让其自动换行
mClContainer.post {
val mClContentWidth = mClContainer.run { width - paddingEnd - paddingLeft }
Copy link
Member

Choose a reason for hiding this comment

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

用了paddingEnd就不要使用paddingLeft，这两个在反布局下是同个东西

@RQ527 RQ527 merged commit f9ab5d3 into develop Dec 11, 2023
3 checks passed
@RQ527 RQ527 deleted the RanSX/Optimize branch December 11, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@985892345 985892345 985892345 approved these changes

@RedSun-V RedSun-V Awaiting requested review from RedSun-V

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RQ527 @985892345