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

Movie module - Show RomHash Region, if not null #1090

Merged
merged 7 commits into from
Feb 10, 2022
Merged

Movie module - Show RomHash Region, if not null #1090

merged 7 commits into from
Feb 10, 2022

Conversation

adelikat
Copy link
Collaborator

@adelikat adelikat commented Feb 9, 2022

Conditionally shows the region, if the rom hash entry is added
The value needs to not be in the title so that it can be styled independently, so I put it all the way to the left. I also only show it if it is relevant, which currently are these values:
J => (JPN),
E => (Europe),
EU => (USA/Europe)

We can easily configure these values if want different examples.

image

image

@meshuggahtas
Copy link
Collaborator

I think (U) USA should be added too

@vadosnaprimer
Copy link
Collaborator

vadosnaprimer commented Feb 9, 2022

I think JPN should be turned into Japan. Alpha-3 codes are used for countries, but we also have Europe and Asia which don't have official alpha-3 codes. So instead of inventing codes for those, I think it would be neater to spell out Japan instead.

Also IMO there should be World and it should be default and hidden, just like any was on the old site. Finally, there should probably also be unknown.

Need opinions from @fsvgm777 and @ThunderAxe31

@adelikat
Copy link
Collaborator Author

adelikat commented Feb 9, 2022

I dont' know what you mean about hidden. The RomHash table has all those things, we are just deciding which to show, and what display value to show.

So it sounds like the only change you want is JPN to Japan?

@vadosnaprimer
Copy link
Collaborator

Hidden I mean not displayed in publication titles.

Also in the case of VVVVVV and possibly other PC games, we need to display numeric game version instead of the region, since region is kinda pointless on PC.

@adelikat
Copy link
Collaborator Author

image

@adelikat
Copy link
Collaborator Author

@vadosnaprimer : I don't think you understand, the fields are in then RomHash table. I'm just formatting and displaying it conditionally. How do you want them displayed is the question

@Masterjun3
Copy link
Collaborator

Here is the discussion in the related issue that wasn't linked yet #906.

TASVideos/Pages/Shared/_MovieModule.cshtml Outdated Show resolved Hide resolved
@adelikat
Copy link
Collaborator Author

Okay, okay, no v. But people need to be putting that v in a lot of rom hash entries then

@vadosnaprimer
Copy link
Collaborator

For cases when the same game works in all regions, we need to have World. USA/Europe doesn't include Japan and PAL Asia. PAL Asia games are rare, but if somebody submits a NES game made there and it uses Dendy mode, that's when we used the Asia region, to indicate it's running in hybrid mode. This is not critical tho.

On the matter of display, I think it's ok to put region and version within the same pair of parentheses.

@Masterjun3 Masterjun3 requested review from Masterjun3 and removed request for Masterjun3 February 10, 2022 16:28
@Masterjun3 Masterjun3 dismissed their stale review February 10, 2022 16:32

One part was fixed but now I have to approve the whole thing? What are you doing Github.

@Masterjun3 Masterjun3 removed their request for review February 10, 2022 16:43
@ThunderAxe31
Copy link
Contributor

I agree that JPN should be Japan, and that any should be World.

I don't like the style of (Japan) (v1.4), as in my opinion it makes it unclear that both things are referring to the game version. I suggest instead using (Japan v1.4).

Also, I want to note that there are more combo cases, like Pokémon Mystery Dungeon, in for which a version is known as USA/Australia.

@adelikat
Copy link
Collaborator Author

JPN has been changed to Japan already.
and neither any nor world show.

@fsvgm777
Copy link
Contributor

fsvgm777 commented Feb 10, 2022

Okay, kinda late to the party, but I agree with hiding any/World, especially for PC games (DOS, Windows, Linux), where region doesn't really matter.

Also, I want to note that there are more combo cases, like Pokémon Mystery Dungeon, in for which a version is known as USA/Australia.

I think that can be put as Other or just USA (if GBA)? Same deal with games released exclusively in Germany (Hugo 2 1/2) or France (no movie examples, but I know of some (e.g. Les Visiteurs)).

I don't like the style of (Japan) (v1.4), as in my opinion it makes it unclear that both things are referring to the game version. I suggest instead using (Japan v1.4).

Agreed, with a comma inbetween region and effective version.

@adelikat
Copy link
Collaborator Author

image

@adelikat adelikat merged commit 075a2cd into main Feb 10, 2022
@adelikat adelikat deleted the pub-region branch February 10, 2022 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants