Hide support button and comments consistently with claim tags #1152
No reviewers
Labels
No labels
android: closed alpha
android: open beta
app-parity
area: devops
area: discovery
area: docs
area: livestream
area: proposal
consider soon
creator
Epic
good first issue
hacktoberfest
help wanted
icebox
Invalid
level: 1
level: 2
level: 3
level: 4
needs: exploration
needs: grooming
needs: priority
needs: repro
needs: tech design
on hold
priority: blocker
priority: high
priority: low
priority: medium
product review
resilience
Tom's Wishlist
type: bug
type: discussion
type: improvement
type: new feature
type: refactor
type: task
type: testing
unplanned
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: LBRYCommunity/lbry-android#1152
Loading…
Reference in a new issue
No description provided.
Delete branch "support-tags-ui-consistency"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
PR Checklist
Please check all that apply to this PR using "x":
PR Type
What kind of change does this PR introduce?
Fixes
Issue Number:1146
What is the current behavior?
See #1146
What is the new behavior?
See #1146
Other information
In comment fragment for channel, the comment form visibility is GONE to avoid it been shown just before being replaced by the texto for channels having comments disabled. This change is not needed for FileViewFragment as this replacement occurs while not in user view.
Two new strings have been added. Text was taken from LBRY.TV. Translation will be needed.
This commit adds code to code which is already been duplicated in various files. This will be fixed in an upcoming PR which replaces comment form with a new widget instead of a layout which should be
Known problem: on content view, the text explaining that comments have been disabled by the author, wallet and rewards bubble is positioned just above it, so the string is not visible on its full length. It will fixed for #1054 very shortly.
Looks good. Just one minor change to make before merging.
Use
View.GONE
here so that the view doesn't occupy any space in the layout.